On Tue, Nov 20, 2012 at 08:39:53PM -0800, John Johansen wrote: > From: John Johansen <[email protected]> > > Move the free_profile fn ahead of aa_alloc_profile so it can be used > in aa_alloc_profile without a forward declaration. > > Signed-off-by: John Johansen <[email protected]> Acked-by: Steve Beattie <[email protected]>
While I don't have a problem with this code re-arrangement, I don't actually see where free_profile() gets used by aa_alloc_profile() or aa_new_null_profile() or anything that looked like a called macro function within them. Attempting to search later patches in the series didn't find anything that added it to those functions. Am I missing something? > --- > security/apparmor/policy.c | 150 > ++++++++++++++++++++++---------------------- > 1 file changed, 75 insertions(+), 75 deletions(-) > > diff --git a/security/apparmor/policy.c b/security/apparmor/policy.c > index 13fc9ef..f4ee72b 100644 > --- a/security/apparmor/policy.c > +++ b/security/apparmor/policy.c -- Steve Beattie <[email protected]> http://NxNW.org/~steve/
signature.asc
Description: Digital signature
-- AppArmor mailing list [email protected] Modify settings or unsubscribe at: https://lists.ubuntu.com/mailman/listinfo/apparmor
