[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PULL 3/5] qemu-bridge-helper: move repeating code in parse
From: |
Jason Wang |
Subject: |
[Qemu-devel] [PULL 3/5] qemu-bridge-helper: move repeating code in parse_acl_file |
Date: |
Mon, 29 Jul 2019 16:33:44 +0800 |
From: Prasad J Pandit <address@hidden>
Move repeating error handling sequence in parse_acl_file routine
to an 'err' label.
Signed-off-by: Prasad J Pandit <address@hidden>
Reviewed-by: Stefan Hajnoczi <address@hidden>
Reviewed-by: Daniel P. Berrangé <address@hidden>
Reviewed-by: Li Qiang <address@hidden>
Signed-off-by: Jason Wang <address@hidden>
---
qemu-bridge-helper.c | 19 +++++++++----------
1 file changed, 9 insertions(+), 10 deletions(-)
diff --git a/qemu-bridge-helper.c b/qemu-bridge-helper.c
index 2058e10..3d50ec0 100644
--- a/qemu-bridge-helper.c
+++ b/qemu-bridge-helper.c
@@ -102,9 +102,7 @@ static int parse_acl_file(const char *filename, ACLList
*acl_list)
if (arg == NULL) {
fprintf(stderr, "Invalid config line:\n %s\n", line);
- fclose(f);
- errno = EINVAL;
- return -1;
+ goto err;
}
*arg = 0;
@@ -121,9 +119,7 @@ static int parse_acl_file(const char *filename, ACLList
*acl_list)
if (!g_str_equal(cmd, "include") && strlen(arg) >= IFNAMSIZ) {
fprintf(stderr, "name `%s' too long: %zu\n", arg, strlen(arg));
- fclose(f);
- errno = EINVAL;
- return -1;
+ goto err;
}
if (strcmp(cmd, "deny") == 0) {
@@ -149,15 +145,18 @@ static int parse_acl_file(const char *filename, ACLList
*acl_list)
parse_acl_file(arg, acl_list);
} else {
fprintf(stderr, "Unknown command `%s'\n", cmd);
- fclose(f);
- errno = EINVAL;
- return -1;
+ goto err;
}
}
fclose(f);
-
return 0;
+
+err:
+ fclose(f);
+ errno = EINVAL;
+ return -1;
+
}
static bool has_vnet_hdr(int fd)
--
2.5.0
- [Qemu-devel] [PULL 0/5] Net patches, Jason Wang, 2019/07/29
- [Qemu-devel] [PULL 1/5] e1000: don't raise interrupt in pre_save(), Jason Wang, 2019/07/29
- [Qemu-devel] [PULL 3/5] qemu-bridge-helper: move repeating code in parse_acl_file,
Jason Wang <=
- [Qemu-devel] [PULL 2/5] qemu-bridge-helper: restrict interface name to IFNAMSIZ, Jason Wang, 2019/07/29
- [Qemu-devel] [PULL 4/5] net: tap: replace snprintf with g_strdup_printf calls, Jason Wang, 2019/07/29
- [Qemu-devel] [PULL 5/5] net/colo-compare.c: Fix memory leak and code style issue., Jason Wang, 2019/07/29
- Re: [Qemu-devel] [PULL 0/5] Net patches, Peter Maydell, 2019/07/29