On Fri, Sep 27, 2013 at 05:17:43PM -0700, Tyler Hicks wrote: > On 2013-09-13 16:52:47, Steve Beattie wrote: > > Finally, a patch of my own. This patch addresses a bunch of the compiler > > string conversion warnings that were introduced with the C++-ification > > patch. > > > > Signed-off-by: Steve Beattie <st...@nxnw.org> > > @@ -284,8 +284,8 @@ extern void free_cond_entry(struct cond_ > > extern void free_cond_list(struct cond_entry *ents); > > extern void print_cond_entry(struct cond_entry *ent); > > extern char *processid(char *string, int len); > > We should probably const the string param in processid() while we're at > it.
Incorporated. > It is safe to get rid of the if statement and just unconditionally free > filename. It is initialized to NULL and now that OPTION_STDOUT does a > strdup(), all options either assign filename using a memory allocation > function or don't assign anything to it at all. > > It is extremely minor and I don't care either way, but I thought I'd > mention it. Incorporated this change as well. > Everything else looks good! > > Acked-by: Tyler Hicks <tyhi...@canonical.com> Thanks! -- Steve Beattie <sbeat...@ubuntu.com> http://NxNW.org/~steve/
signature.asc
Description: Digital signature
-- AppArmor mailing list AppArmor@lists.ubuntu.com Modify settings or unsubscribe at: https://lists.ubuntu.com/mailman/listinfo/apparmor