On Mon, Jan 25, 2016 at 12:07:13PM -0800, Seth Arnold wrote:
> On Mon, Jan 25, 2016 at 11:47:09AM -0800, Steve Beattie wrote:
> > This is an unchanged resend of the patch from this thread:
> > https://lists.ubuntu.com/archives/apparmor/2015-August/008494.html
> > 
> > The first entry in the grouping_count array is never initialized to 0;
> > subsequent depths are. This patch fixes the issue.
> > 
> > [parser-fix_uninitialized_grouping_value.patch]
> > 
> > Signed-off-by: Steve Beattie <[email protected]>
> 
> Acked-by: Seth Arnold <[email protected]>
> 
> (slighly suggested to memset the whole thing to zero, or maybe assign the
> whole thing to zero:  int grouping_count[MAX_ALT_DEPTH] = {0}; -- but
> this alone is better than current. :)

Yes, much better suggestion, thanks. I'll incorporate that into the
commit.


-- 
Steve Beattie
<[email protected]>
http://NxNW.org/~steve/

Attachment: signature.asc
Description: Digital signature

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

Reply via email to