The debugging code for profile entries contains a check to ensure that it's not NULL, but the list iterator macro already ensures that the iteration will stop if the item is NULL, making the check redundant.
Coverity CID #55983 Signed-off-by: Steve Beattie <[email protected]> --- parser/parser_misc.c | 4 ---- 1 file changed, 4 deletions(-) Index: b/parser/parser_misc.c =================================================================== --- a/parser/parser_misc.c +++ b/parser/parser_misc.c @@ -665,9 +665,6 @@ void debug_cod_entries(struct cod_entry printf("--- Entries ---\n"); list_for_each(list, item) { - if (!item) - printf("Item is NULL!\n"); - printf("Mode:\t"); if (HAS_CHANGE_PROFILE(item->mode)) printf(" change_profile"); -- Steve Beattie <[email protected]> http://NxNW.org/~steve/
signature.asc
Description: PGP signature
-- AppArmor mailing list [email protected] Modify settings or unsubscribe at: https://lists.ubuntu.com/mailman/listinfo/apparmor
