From c437e9d4a5a68cb3449db9b5111453e11bf00818 Mon Sep 17 00:00:00 2001 From: Christian Boltz Date: Fri, 13 Jul 2018 15:21:24 +0200 Subject: [PATCH] add missing 'break' in load_profile() 'case OPTION_OFILE' missed the 'break', which means if did fallthrough to the default case. Adding the 'break' means no longer executing another PERROR, and no longer executing the 'exit(1)' in the default branch. References: coverity #55994 --- parser/parser_interface.c | 1 + 1 file changed, 1 insertion(+) diff --git a/parser/parser_interface.c b/parser/parser_interface.c index 5d9e0a0a2..07ea0f6db 100644 --- a/parser/parser_interface.c +++ b/parser/parser_interface.c @@ -118,6 +118,7 @@ int load_profile(int option, aa_kernel_interface *kernel_interface, case OPTION_OFILE: PERROR(_("%s: Unable to write to output file\n"), progname); + break; default: PERROR(_("%s: ASSERT: Invalid option: %d\n"), progname, option);