Fix: consumer: assert that stream chunk != NULL (not channel)
authorMathieu Desnoyers <mathieu.desnoyers@efficios.com>
Fri, 9 Aug 2019 16:25:57 +0000 (12:25 -0400)
committerJérémie Galarneau <jeremie.galarneau@efficios.com>
Thu, 5 Sep 2019 20:39:15 +0000 (16:39 -0400)
commitcd91c17db6e8d5a8aabf5a7533d945925babe0e6
treeacea8e1201f32850a6a02f6b99cbe4e9fb62d038
parenta57bb1b0809f64ee342883598f92555158ceca22
Fix: consumer: assert that stream chunk != NULL (not channel)

This assert sometimes triggers in
tests/regression/tools/crash/test_crash: test_shm_path_per_pid_sigint
because the destroy is performed when there is still unflushed data in
the buffers. The destroy performs a chunk close on the consumer daemon,
which effectively sets the channel chunk to NULL, while there are still
references to the chunk in the streams (which are still active).

Change the assertion in the stream read to validate that stream chunk !=
NULL instead.

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>
src/common/consumer/consumer.c
This page took 0.024631 seconds and 4 git commands to generate.