Hello,

Am Freitag, 12. Mai 2017, 00:58:32 CEST schrieb Tyler Hicks:
...
> diff --git a/parser/parser_main.c b/parser/parser_main.c
> index 80c243d..8f1af4f 100644
> --- a/parser/parser_main.c
> +++ b/parser/parser_main.c
> @@ -1159,6 +1159,7 @@ int main(int argc, char *argv[])
>                       continue;
> 
>               if (profilename && stat(profilename, &stat_file) == -1) {
> +                     last_error = errno;
>                       PERROR("File %s not found, skipping...\n", profilename);

IIRC I've seen this message also for
    apparmor_parser /no/such/directory/
so maybe you should change or simply remove the word "File" ;-)
(yes, that's unrelated to this patch, so feel free to commit this patch 
as is)

I didn't test v2, but the changes since v1 look like an improvement to 
me ;-)


Regards,

Christian Boltz

PS: non-random sig - translated:
        Unix: Everything is a file. If something isn't a file, it kindly 
        has to cover itsself as such
-- 
Unix: Alles ist ein File, und was kein File ist, hat sich gefaelligst
als ein solches zu tarnen.      [Wolfgang Weisselberg in linux-liste]

Attachment: signature.asc
Description: This is a digitally signed message part.

-- 
AppArmor mailing list
[email protected]
Modify settings or unsubscribe at: 
https://lists.ubuntu.com/mailman/listinfo/apparmor

Reply via email to