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: lslebodn(a)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