[kronosnet/kronosnet] 6d1625: [build] workaround memory alignment bug in nss in ...

Branch: refs/heads/master Home: https://github.com/kronosnet/kronosnet Commit: 6d1625538adedb911d47e6b2047573ee85780b2c https://github.com/kronosnet/kronosnet/commit/6d1625538adedb911d47e6b204757… Author: Fabio M. Di Nitto <fdinitto(a)redhat.com> Date: 2017-11-15 (Wed, 15 Nov 2017) Changed paths: M build-aux/check.mk Log Message: ----------- [build] workaround memory alignment bug in nss in combination with valgrind on Linux/i386 Over the past few weeks, we noticed CI failing on i386 when running make check-memcheck (test suite executed with valgring). Those failures were not consistent and sometime random. After some heavy debugging it turns out that those failures are a combination of libnss3 internal memory allocator bug and valgrind internal memory allocator. Current libnss3 memory allocator expects a 16 bytes memory aligned buffer and in the event the memory is not aligned, it would round it. The issue is that, in performing this rounding, nss does not track properly the original address of the memory, and the equivalent of subsequent free of this buffer would fail, leaking memory and potentially accessing memory not allocated or corrpting memory. The nss bug is already fixed upstream by commit: changeset: 13557:52e38f913220 Valgrind on Linux/i386 memory allocator can return memory that is not aligned to the 16 bytes, contrary to what malloc/glibc does. Valgrind was returning (somehow randomly) a non aligned buffer to nss triggering the nss bug and the leak. By forcing valgrind to use aligned memory, nss bug does not trigger and there is no memory leak during the execution of the test suite. The memory leak never triggered when running knet in normal conditions. Signed-off-by: Fabio M. Di Nitto <fdinitto(a)redhat.com> Commit: bc519d4a4a5f7c9a6cadae0453f15cc2ff432960 https://github.com/kronosnet/kronosnet/commit/bc519d4a4a5f7c9a6cadae0453f15… Author: Fabio M. Di Nitto <fdinitto(a)redhat.com> Date: 2017-11-15 (Wed, 15 Nov 2017) Changed paths: M build-aux/check.mk Log Message: ----------- Merge pull request #88 from kronosnet/nss-workaround [build] workaround memory alignment bug in nss in combination with va… Compare: https://github.com/kronosnet/kronosnet/compare/8156886888aa...bc519d4a4a5f