From: Thomas Hindoe Paaboel Andersen Date: Fri, 3 Oct 2014 01:58:51 +0000 (+0200) Subject: test-barrier: add checks after the barrier constructor X-Git-Tag: v217~352 X-Git-Url: http://www.chiark.greenend.org.uk/ucgi/~ianmdlvl/git?a=commitdiff_plain;h=2ad8887a12aeff9108606bb31e1557103a3b95df;p=elogind.git test-barrier: add checks after the barrier constructor Coverity seems to think that we can later end up with the "them" fd having a negative value. Even after a succesful barrier_create. Add some test to verify that the constructor went well. If coverity still complains then it must mean that it thinks the the value is overwritten later. --- diff --git a/src/test/test-barrier.c b/src/test/test-barrier.c index 36f27809c..cb75f7314 100644 --- a/src/test/test-barrier.c +++ b/src/test/test-barrier.c @@ -64,6 +64,10 @@ static void sleep_for(usec_t usecs) { pid_t pid1, pid2; \ \ assert_se(barrier_create(&b) >= 0); \ + assert_se(b.me > 0); \ + assert_se(b.them > 0); \ + assert_se(b.pipe[0] > 0); \ + assert_se(b.pipe[1] > 0); \ \ pid1 = fork(); \ assert_se(pid1 >= 0); \