Opened 6 years ago

Closed 6 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 6 years ago.

Download all attachments as: .zip

Change History (2)

Changed 6 years ago by sjamaan

Attachment: move-memory.patch added

comment:1 Changed 6 years ago by sjamaan

Resolution: fixed
Status: newclosed

Fixed by e29b85b242178e2b9b3317a3e8031af76dce2316

Note: See TracTickets for help on using tickets.