On Thu, Apr 03, 2014 at 05:13:19PM +0200, Jakub Hrozek wrote: > On Thu, Apr 03, 2014 at 01:25:04PM +0200, Sumit Bose wrote: > > On Thu, Apr 03, 2014 at 12:58:03PM +0200, Jakub Hrozek wrote: > > > On Thu, Apr 03, 2014 at 09:45:46AM +0200, Sumit Bose wrote: > > > > On Wed, Apr 02, 2014 at 10:29:15PM +0200, Jakub Hrozek wrote: > > > > > Hi, > > > > > > > > > > please see the attached patch that fixes a use-after-free bug in the > > > > > SELinux provider. > > > > > > > > > > Pair-Programmed-With: lsleb...@redhat.com > > > > > > > > The patch solves the issue, but I think the issue was there in the first > > > > place, because the 'order' string is somewhat hidden. I would suggest to > > > > use a struct like > > > > > > > > struct order_ctx { > > > > const char *order; > > > > char **order_array; > > > > size_t order_count; > > > > }; > > > > > > > > It might be a bit overkill for this small usage, but I think it makes > > > > the intention and the relation between order and order_array more clear. > > > > > > > > bye, > > > > Sumit > > > > > > Would you accept this small patch for 1.11 and your proposed (and > > > better, I agree) change for master? > > > > sure, ACK. > > > > But I think it would be better to apply this one to 1.11 and master and > > then add another patch with the changes to master. > > > > bye, > > Sumit > > OK, thank you for the acks. I filed another ticket for the refactor and > gave it to Michal: > https://fedorahosted.org/sssd/ticket/2304
Pushed upstream: master: 355b8a655cfcc4e783077d12f76b55da1d23fb87 sssd-1-11: ac93a2d27415abd730aa1063b1689def8be9dbe9 _______________________________________________ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/sssd-devel