summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMatthias Melcher <github@matthiasm.com>2023-01-29 13:49:37 +0100
committerMatthias Melcher <github@matthiasm.com>2023-01-29 13:49:37 +0100
commit16af5f432e9f7cc62b7205d47c147e1fc5909e94 (patch)
tree25538ed608e6d91ab68b5bc0a203e4d09a76dc94
parent85ac3d31474e48d0e84d2dd03b369e0e559e007e (diff)
Avoid Linux warning
-rw-r--r--fluid/ExternalCodeEditor_UNIX.cxx7
1 files changed, 5 insertions, 2 deletions
diff --git a/fluid/ExternalCodeEditor_UNIX.cxx b/fluid/ExternalCodeEditor_UNIX.cxx
index 2a8f8a513..e1e2c0e60 100644
--- a/fluid/ExternalCodeEditor_UNIX.cxx
+++ b/fluid/ExternalCodeEditor_UNIX.cxx
@@ -402,7 +402,9 @@ int ExternalCodeEditor::start_editor(const char *editor_cmd,
execvp(args[0], args); // run command - doesn't return if succeeds
if (alert_pipe_open_) {
int err = errno;
- ::write(alert_pipe_[1], &err, sizeof(int));
+ if (::write(alert_pipe_[1], &err, sizeof(int)) != sizeof(int)) {
+ // should not happen, but if it does, at least we tried
+ }
}
exit(1);
}
@@ -555,7 +557,8 @@ int ExternalCodeEditor::editors_open() {
void ExternalCodeEditor::alert_pipe_cb(FL_SOCKET s, void* d) {
ExternalCodeEditor* self = (ExternalCodeEditor*)d;
self->last_error_ = 0;
- ::read(s, &self->last_error_, sizeof(int));
+ if (::read(s, &self->last_error_, sizeof(int)) != sizeof(int))
+ return;
const char* cmd = self->command_line_.value();
if (cmd && *cmd) {
if (cmd[0] == '/') { // is this an absoluet filename?