ctdb-tests: Pretend not to ignore return from fgets()

With some compilers and picky compiler settings, ignoring the return
value of fgets() can cause compilation to fail.

Signed-off-by: Martin Schwenke <martin@meltin.net>
Reviewed-by: Amitay Isaacs <amitay@gmail.com>

diff --git a/ctdb/tests/src/fake_ctdbd.c b/ctdb/tests/src/fake_ctdbd.c
index 143acf87678d823981c23cc19c02d417a0440a0b..35bca88b28633d2b772a0e09fd90e67949401a56 100644 (file)
--- a/ctdb/tests/src/fake_ctdbd.c
+++ b/ctdb/tests/src/fake_ctdbd.c
@@ -568,8 +568,13 @@ static bool reclock_parse(struct ctdbd_context *ctdb)
                goto fail;
        }
 ok:
-       /* Swallow possible blank line following section */
-       fgets(line, sizeof(line), stdin);
+       /* Swallow possible blank line following section.  Picky
+        * compiler settings don't allow the return value to be
+        * ignored, so make the compiler happy.
+        */
+       if (fgets(line, sizeof(line), stdin) == NULL) {
+               ;
+       }
        DEBUG(DEBUG_INFO, ("Parsing reclock done\n"));
        return true;