Skip to content

add missing 'break' in load_profile()

Christian Boltz requested to merge cboltz/apparmor:cboltz-parser-break into master

'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

Note: I'm not an expert on the parser code, so please think twice before accepting this ;-)

Merge request reports