Opened 10 years ago

Closed 10 years ago

#1136 closed defect (fixed)

move-memory! bug similar to #1135

Reported by: sjamaan Owned by:
Priority: major Milestone: 4.10.0
Component: core libraries Version: 4.9.x
Keywords: Cc:
Estimated difficulty:

Description

We need to check all the uses of memcpy, to see if there are any left which are wrong and should really be memmove due to the possibility of overlapping memory regions.

Attached is a patch which adds a test, which somehow succeeds with the current code. However, by adding this test I found another bug in the types.db code: it uses w2b which doesn't exist in chicken.h but only in lolevel.scm. This macro does exactly the same as C_bytes, so I've rewritten it to that.

Attachments (1)

move-memory.patch​ (2.4 KB) - added by sjamaan 10 years ago.

Download all attachments as: .zip

Change History (2)

Changed 10 years ago by sjamaan

Attachment: move-memory.patch​ added

comment:1 Changed 10 years ago by sjamaan

Resolution: → fixed
Status: new → closed

Fixed by e29b85b242178e2b9b3317a3e8031af76dce2316

Note: See TracTickets for help on using tickets.