diff options
| author | Rafał Miłecki | 2019-09-05 21:07:21 +0000 |
|---|---|---|
| committer | Rafał Miłecki | 2019-09-05 21:07:21 +0000 |
| commit | 0bcbbbf1abf76bf3aa8af5657308cac61f7de602 (patch) | |
| tree | a0ee20fa248d709638c234f3d6a332d2bc0d6a98 | |
| parent | 34ac88cb873ad81f1510e2207c34b6abaa1ae16a (diff) | |
| download | procd-0bcbbbf1abf76bf3aa8af5657308cac61f7de602.tar.gz | |
system: fix uninitialized variables in firmware validation code
This fixes:
system.c: In function 'validate_firmware_image':
system.c:403:6: error: 'fd' may be used uninitialized in this function [-Werror=maybe-uninitialized]
if (fd >= 0) {
^
system.c:446:4: error: 'jsobj' may be used uninitialized in this function [-Werror=maybe-uninitialized]
blobmsg_add_object(&b, jsobj);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Fixes: e990e215e8a3 ("system: add "validate_firmware_image" ubus method")
Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
| -rw-r--r-- | system.c | 3 |
1 files changed, 2 insertions, 1 deletions
@@ -386,8 +386,8 @@ static int proc_signal(struct ubus_context *ctx, struct ubus_object *obj, static int validate_firmware_image_call(const char *file) { const char *path = "/usr/libexec/validate_firmware_image"; + json_object *jsobj = NULL; json_tokener *tok; - json_object *jsobj; char buf[64]; ssize_t len; int fds[2]; @@ -402,6 +402,7 @@ static int validate_firmware_image_call(const char *file) return -errno; case 0: /* Set stdin & stderr to /dev/null */ + fd = open("/dev/null", O_RDWR); if (fd >= 0) { dup2(fd, 0); dup2(fd, 2); |