use memcpy on unaligned pointer to avoid crash on armv7 (BUS_ADRALN)

[hgourvest]

There is a crash in zhash_pack method when writting the 4 bytes corresponding to value length, to unaligned pointer address, this occur on Android armv7. I solve the problem using memcpy instead.

[hgourvest]

probably same problem on zlistx_pack and zhashx_pack_own

[sappo]

Thanks for your contribution :)

memcopy has been used before. Due to your refactoring I find it hard to get the actual change. Can you please point me to the correct code lines.

P. S. for future commits please use the problem solution Formular described in our contribution and style guidelines. Also see the commit history for examples

[sappo]

Ah got it after all.

[hgourvest]

[zhash] before: 704 after: 706
[zhashx] before: 822 after: 824
[zlistx] before: 674 after: 675

sorry for the spaces, Android studio automatically remove unncecessary space :/

[sappo]

Thanks. Don't worry about the spaces my vim configuration does the same thing ;) Am Fr., 30. Nov. 2018, 11:40 hat Henri Gourvest <notifications@github.com> geschrieben:

…

[zhash] before: 704 after: 706 [zhashx] before: 822 after: 824 [zlistx] before: 674 after: 675 sorry for the spaces, Android studio automatically remove unncecessary space :/ — You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub <#1959 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAeGuqkAyok01WghdZSYzeu6r2cTtYPTks5u0QsQgaJpZM4Y6xLG> .