search

GemTalk / PharoGemStoneFFI

GemStone GCI access via Pharo FFI
MIT License
1 stars 3 forks source link

GsExternalByteString>>asByteString leaks memory #10

Closed martinmcclure closed 3 years ago

martinmcclure commented 3 years ago

GsExternalByteString>>asByteString is implemented as

^ ByteString withAll: (self copyUpTo: Character null)

The copyUpTo: creates another instance of GsExternalByteString, which is never freed.

martinmcclure commented 3 years ago

Candidate fix in commit 333e047d.

martinmcclure commented 3 years ago

Seems to test OK, so pushed to development and main.