Fix some warn_unused_result warnings

While building on gcc 9.3.0
flatpak · Oct 12, 2021 · c6845ad · c6845ad
1 parent 6ebe331
commit c6845ad
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 3 deletions.
diff --git a/common/flatpak-utils.c b/common/flatpak-utils.c
@@ -8060,13 +8060,25 @@ flatpak_get_cursor_pos (int * row, int *col)
 void
 flatpak_hide_cursor (void)
 {
- write (STDOUT_FILENO, FLATPAK_ANSI_HIDE_CURSOR, strlen (FLATPAK_ANSI_HIDE_CURSOR));
+ const size_t flatpak_hide_cursor_len = strlen (FLATPAK_ANSI_HIDE_CURSOR);
+ const ssize_t write_ret = write (STDOUT_FILENO, FLATPAK_ANSI_HIDE_CURSOR,
+ flatpak_hide_cursor_len);
+
+ if (write_ret < 0)
+ g_warning ("write() failed: %zd = write(STDOUT_FILENO, FLATPAK_ANSI_HIDE_CURSOR, %zu)",
+ write_ret, flatpak_hide_cursor_len);
 }
 
 void
 flatpak_show_cursor (void)
 {
- write (STDOUT_FILENO, FLATPAK_ANSI_SHOW_CURSOR, strlen (FLATPAK_ANSI_SHOW_CURSOR));
+ const size_t flatpak_show_cursor_len = strlen (FLATPAK_ANSI_SHOW_CURSOR);
+ const ssize_t write_ret = write (STDOUT_FILENO, FLATPAK_ANSI_SHOW_CURSOR,
+ flatpak_show_cursor_len);
+
+ if (write_ret < 0)
+ g_warning ("write() failed: %zd = write(STDOUT_FILENO, FLATPAK_ANSI_SHOW_CURSOR, %zu)",
+ write_ret, flatpak_show_cursor_len);
 }
 
 void

diff --git a/revokefs/demo.c b/revokefs/demo.c
@@ -74,7 +74,11 @@ main (int argc, char *argv[])
  }
 
  g_print ("Started revokefs, press enter to revoke");
- fgets(buf, sizeof(buf), stdin);
+ if (!fgets(buf, sizeof(buf), stdin))
+ {
+ perror ("fgets");
+ }
+
  g_print ("Revoking write permissions");
  shutdown (sockets[1], SHUT_RDWR);
 }