diff options
author | Phaedrus Leeds <mwleeds@protonmail.com> | 2021-10-12 14:31:43 -0700 |
---|---|---|
committer | GitHub <noreply@github.com> | 2021-10-12 14:31:43 -0700 |
commit | 4f6c13e7511dcef2965ed2c1d1a29982427fed0a (patch) | |
tree | c400d03ddba0a8acbc3e9a3339e23b8827b295cd | |
parent | 0a8d217750c6b0709bf774adf5e366d7357dc6bc (diff) | |
parent | c6845ad70b757904d84210c5b613b36f3d934eeb (diff) | |
download | flatpak-4f6c13e7511dcef2965ed2c1d1a29982427fed0a.tar.gz |
Merge pull request #4327 from ericcurtin/warn_unused_result
Fix some warn_unused_result warnings
-rw-r--r-- | common/flatpak-utils.c | 16 | ||||
-rw-r--r-- | revokefs/demo.c | 6 |
2 files changed, 19 insertions, 3 deletions
diff --git a/common/flatpak-utils.c b/common/flatpak-utils.c index 3824139a..7674e0be 100644 --- a/common/flatpak-utils.c +++ b/common/flatpak-utils.c @@ -8396,13 +8396,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 index 6608b52f..6685314f 100644 --- 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); } |