On Thu, May 09, 2013 at 02:58:40AM -0700, John Johansen wrote:
> >> -static struct aa_fs_entry aa_fs_entry =
> >> - AA_FS_DIR("apparmor", aa_fs_entry_apparmor);
> >> +static struct aa_fs_entry aa_fs_entry[] = {
> >> + AA_FS_DIR("apparmor", aa_fs_entry_apparmor),
> >> + { }
> >> +};
> >
> > I didn't see the reasoning behind this change.. nothing really wrong
> > with it, but it appeared to just complicate nearly every line of code
> > that touched aa_fs_entry, except one..
> >
> the { } entry is needed otherwise we can walk off the end of the array in
> some of the loop code. ie its a necessary terminatorExcept I haven't spotted the iteration over the one-element-array yet.. :) Thanks
signature.asc
Description: Digital signature
-- AppArmor mailing list [email protected] Modify settings or unsubscribe at: https://lists.ubuntu.com/mailman/listinfo/apparmor
