Summary

move-memory! bug similar to #1135

Metadata

• Id: 039206d1fcaa2b4561fdab8ab5d1fbe7c9cd5bde
• Trac id: 1136
• Type: defect
• Reporter: sjamaan
• Owner:
• Cc:
• Status: closed
• Component: core libraries
• Estimated difficulty:
• Resolution: fixed
• Priority: major
• Milestone: 4.10.0
• Version: 4.9.x
• Changetime: 2014-08-03 18:33:54 UTC
• Created: 2014-06-30 21:25:16 UTC
• Keywords:

Attachments

• 039206d1fcaa2b4561fdab8ab5d1fbe7c9cd5bde/attachments/move-memory.patch

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.

Changes and comments

[2014-06-30 21:25:23 UTC] sjamaan attached move-memory.patch (description=#f)

[2014-08-03 18:33:54 UTC] sjamaan changed status from new to closed

[2014-08-03 18:33:54 UTC] sjamaan set resolution to fixed

[2014-08-03 18:33:54 UTC] sjamaan wrote:

Fixed by e29b85b242178e2b9b3317a3e8031af76dce2316