properly handle return code of pipe() syscall

[project/ubox.git] / log / logd.c

diff --git a/log/logd.c b/log/logd.c
index 5c614f6..8cebeb8 100644 (file)
--- a/log/logd.c
+++ b/log/logd.c
@@ -58,12 +58,11 @@ client_close(struct ustream *s)
 static void
 client_notify_write(struct ustream *s, int bytes)
 {
-       client_close(s);
 }
 
 static void client_notify_state(struct ustream *s)
 {
-       return client_close(s);
+       client_close(s);
 }
 
 static int
@@ -83,7 +82,10 @@ read_log(struct ubus_context *ctx, struct ubus_object *obj,
                        count = blobmsg_get_u32(tb);
        }
 
-       pipe(fds);
+       if (pipe(fds) == -1) {
+               fprintf(stderr, "logd: failed to create pipe: %s\n", strerror(errno));
+               return -1;
+       }
        ubus_request_set_fd(ctx, req, fds[0]);
        cl = calloc(1, sizeof(*cl));
        cl->s.stream.notify_write = client_notify_write;
@@ -165,8 +167,10 @@ ubus_connect_handler(struct ubus_context *ctx)
        int ret;
 
        ret = ubus_add_object(ctx, &log_object);
-       if (ret)
+       if (ret) {
                fprintf(stderr, "Failed to add object: %s\n", ubus_strerror(ret));
+               exit(1);
+       }
        fprintf(stderr, "log: connected to ubus\n");
 }