[apparmor] [patch] [parser] allow the parser to process directories as a parameter
Christian Boltz
apparmor at cboltz.de
Sun Sep 29 11:06:29 UTC 2013
Hello,
Am Sonntag, 29. September 2013 schrieb John Johansen:
> allow directories to be passed to the parser
>
> Allow directories to be passed directly to the parser and handled
> instead of needing an initscript to find the files in the directory.
Sounds like a good idea :-)
> Signed-off-by: John Johansen <john.johansen at canonical.com>
> --- a/parser/parser_main.c
> +++ b/parser/parser_main.c
> @@ -1106,6 +1106,48 @@ out:
> return retval;
> }
>
> +/* data - name of parent dir */
> +static int profile_dir_cb(__unused DIR *dir, const char *name, struct
> stat *st,
It may be a silly question, but: what's the purpose of "__unused DIR
*dir"? It looks, well, unused (and therefore superfluous) to me ;-)
> +/* data - name of parent dir */
> +static int binary_dir_cb(__unused DIR *dir, const char *name, struct
> stat *st,
Same silly question about "__unused DIR *dir" here ;-)
> diff --git a/parser/parser_misc.c b/parser/parser_misc.c
> index 67a7779..e3b9d6a 100644
> --- a/parser/parser_misc.c
> +++ b/parser/parser_misc.c
> @@ -51,6 +51,47 @@
> +static struct ignored_suffix_t ignored_suffixes[] = {
> + /* Debian packging files, which are in flux during install
> + should be silently ignored. */
> + { ".dpkg-new", 9, 1 },
> + { ".dpkg-old", 9, 1 },
> + { ".dpkg-dist", 10, 1 },
> + { ".dpkg-bak", 9, 1 },
> + /* RPM packaging files have traditionally not been silently
> + ignored */
> + { ".rpmnew", 7, 0 },
> + { ".rpmsave", 8, 0 },
> + /* Backup files should be mentioned */
> + { "~", 1, 0 },
> + { NULL, 0, 0 }
> +};
Should we also ignore *.orig and *.rej (from recected patches)?
(They should be mentioned.)
Regards,
Christian Boltz
--
>Ein wenig künstlerische Freiheit darf doch wohl noch sein, oder?
Nur, falls Du die Artistic Licence unterschrieben hast.
[Bernd Brodesser und Martin Leidig in suse-talk]
More information about the AppArmor
mailing list