Fix for crash bug in memcpy

Arachnid / solidity-stringutils

Basic string utilities for Solidity

Apache License 2.0

1.14k stars 373 forks source link

Fix for crash bug in memcpy #49

Open jj0b opened 2 years ago

jj0b commented 2 years ago

If toString is run on an exactly 32 character slice, memcpy will crash. It took me more hours than I care to admit to find and fix this. Hope this can be merged or at least that this PR will serve as a warning to anyone else who may run into this. :)

RyanRHall commented 2 years ago

@jj0b can you add a test that demonstrates the bug?