tests/qtest/ipmi-kcs: Fix assert side-effect
authorPhilippe Mathieu-Daudé <philmd@redhat.com>
Wed, 2 Sep 2020 08:07:57 +0000 (10:07 +0200)
committerThomas Huth <thuth@redhat.com>
Thu, 3 Sep 2020 10:47:33 +0000 (12:47 +0200)
commit978382b453b0e40cc17a983b8c1676386fd3cc5e
tree67e59816540b657aa9fb40768bdff073b7abc096
parented943cc934a82fd78102f0df2084a8f3c556716f
tests/qtest/ipmi-kcs: Fix assert side-effect

Fix assert side-effect reported by Coverity:

  /qemu/tests/qtest/ipmi-kcs-test.c: 84 in kcs_wait_obf()
  83         while (IPMI_KCS_CMDREG_GET_OBF() == 0) {
  >>>     CID 1432368:  Incorrect expression  (ASSERT_SIDE_EFFECT)
  >>>     Argument "--count" of g_assert() has a side effect.  The containing function might work differently in a non-debug build.
  84             g_assert(--count != 0);

Reported-by: Coverity (CID 1432368)
Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Message-Id: <20200902080801.160652-2-philmd@redhat.com>
Reviewed-by: Thomas Huth <thuth@redhat.com>
Signed-off-by: Thomas Huth <thuth@redhat.com>
tests/qtest/ipmi-kcs-test.c