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)
Change History (2)
Changed 10 years ago by
Attachment: | move-memory.patch added |
---|
comment:1 Changed 10 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
Note: See
TracTickets for help on using
tickets.
Fixed by e29b85b242178e2b9b3317a3e8031af76dce2316