Skip to content

Conversation

bluca
Copy link
Member

@bluca bluca commented Aug 9, 2017

Solutions: see commits

bluca added 7 commits August 8, 2017 20:44
Solution: rename to test_randof
Solution: add zstr_send[m]_compress and zstr_recv_compress to the API
and add dependency to lz4
Solution: assert if zmq_msg_init_size returns an error, which can
happen when size > 33 bytes due to a malloc failure (OOM).
Otherwise the following memcopy will write on random memory.
Solution: install libsystemd-dev and liblz4-dev.
Install lz4 on OSX only for the legacy builds as an optimisation.
Running brew update takes many minutes and slows down the build
considerably, so only do it where it's already been done as the
legacy builds require libsodium.
@vyskocilm
Copy link
Contributor

How did you managed to add 1024 lines? 😀

@vyskocilm vyskocilm merged commit b7c8a8b into zeromq:master Aug 9, 2017
@bluca
Copy link
Member Author

bluca commented Aug 9, 2017

I'm just that good :-D

@bluca bluca deleted the compression branch August 9, 2017 15:59
@bluca
Copy link
Member Author

bluca commented Aug 9, 2017

@jimklimov FYI on the test_random rename - cmake/msvc were not happy with the "-" in the name for some reasons

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants