diff options
| author | Andreas Schneider <asn@cryptomilk.org> | 2023-04-21 11:30:31 +0200 |
|---|---|---|
| committer | Andreas Schneider <asn@cryptomilk.org> | 2023-04-21 12:14:48 +0200 |
| commit | 4d654472e65ed28b7cd9e66c91d98b9991452266 (patch) | |
| tree | 3f1af26c48eb8408d9f696820a414c35e1330c71 /src/nvim/ui_client.c | |
| parent | 54f5602038975b28570f4ab183b3388842347f57 (diff) | |
| download | rneovim-4d654472e65ed28b7cd9e66c91d98b9991452266.tar.gz rneovim-4d654472e65ed28b7cd9e66c91d98b9991452266.tar.bz2 rneovim-4d654472e65ed28b7cd9e66c91d98b9991452266.zip | |
fix(ui_client): check return code of dup()
gsrc/nvim/ui_client.c: In function ‘ui_client_start_server’:
gsrc/nvim/ui_client.c:68:5: warning: ignoring return value of ‘dup’ declared with attribute ‘warn_unused_result’ [-Wunused-result]
68 | dup(stderr_isatty ? STDERR_FILENO : STDOUT_FILENO);
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Diffstat (limited to 'src/nvim/ui_client.c')
| -rw-r--r-- | src/nvim/ui_client.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/src/nvim/ui_client.c b/src/nvim/ui_client.c index b93b31f7dc..e177c0a60d 100644 --- a/src/nvim/ui_client.c +++ b/src/nvim/ui_client.c @@ -65,7 +65,11 @@ uint64_t ui_client_start_server(int argc, char **argv) #ifdef MSWIN os_open_conin_fd(); #else - dup(stderr_isatty ? STDERR_FILENO : STDOUT_FILENO); + int fd = dup(stderr_isatty ? STDERR_FILENO : STDOUT_FILENO); + if (fd < 0) { + return 0; + } + // FIXME: resource leak of fd #endif } |