search

GemTalk / PharoGemStoneFFI

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

executeStringAndFetchResultString: can answer extra characters #9

Closed martinmcclure closed 3 years ago

martinmcclure commented 3 years ago

Reported by @chisandrei. GciThreadSafeInterface>>executeStringAndFetchResultString: sometimes answers a string with a variable number of extra characters after the correct response.

This method uses GsExternalByteString>>asByteString to construct the result string. This method uses all bytes in the receiver up to the first null, rather than the byte count returned by GciTsExecuteFetchBytes.

I'm working on a fix, which looks relatively simple.

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.