@@ -74,11 +74,12 @@ static int parse_acl_file(const char *filename, ACLList *acl_list)
{
FILE *f;
char line[4096];
+ int ret = -EINVAL;
ACLRule *acl_rule;
f = fopen(filename, "r");
if (f == NULL) {
- return -1;
+ return -errno;
}
while (fgets(line, sizeof(line), f) != NULL) {
@@ -102,9 +103,8 @@ 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;
+ ret = -EINVAL;
+ goto failure;
}
*arg = 0;
@@ -142,15 +142,17 @@ 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;
+ ret = -EINVAL;
+ goto failure;
}
}
+ ret = 0;
+
+failure:
fclose(f);
- return 0;
+ return ret;
}
static bool has_vnet_hdr(int fd)
@@ -238,7 +240,7 @@ int main(int argc, char **argv)
ACLRule *acl_rule;
ACLList acl_list;
int access_allowed, access_denied;
- int ret = EXIT_SUCCESS;
+ int ret;
#ifdef CONFIG_LIBCAP
/* if we're run from an suid binary, immediately drop privileges preserving
@@ -272,9 +274,10 @@ int main(int argc, char **argv)
/* parse default acl file */
QSIMPLEQ_INIT(&acl_list);
- if (parse_acl_file(DEFAULT_ACL_FILE, &acl_list) == -1) {
- fprintf(stderr, "failed to parse default acl file `%s'\n",
- DEFAULT_ACL_FILE);
+ ret = parse_acl_file(DEFAULT_ACL_FILE, &acl_list);
+ if (ret < 0) {
+ fprintf(stderr, "failed to parse default acl file `%s': %s\n",
+ DEFAULT_ACL_FILE, strerror(ret));
ret = EXIT_FAILURE;
goto cleanup;
}
@@ -416,6 +419,7 @@ int main(int argc, char **argv)
/* ... */
/* profit! */
+ ret = EXIT_SUCCESS;
cleanup:
Handle errors and cleanup from the error in a unified place for parse_acl_file(). Signed-off-by: Doug Goldstein <cardoe@cardoe.com> CC: Anthony Liguori <aliguori@us.ibm.com> CC: Richa Marwaha <rmarwah@linux.vnet.ibm.com> CC: Corey Bryant <coreyb@linux.vnet.ibm.com> TO: qemu-devel@nongnu.org --- qemu-bridge-helper.c | 28 ++++++++++++++++------------ 1 file changed, 16 insertions(+), 12 deletions(-)