[SSSD] [PATCHES] check return value

Sumit Bose sbose at redhat.com
Tue Apr 8 11:00:57 UTC 2014


On Mon, Apr 07, 2014 at 06:13:53PM +0200, Sumit Bose wrote:
> On Mon, Apr 07, 2014 at 02:22:04PM +0200, Pavel Reichl wrote:
> > Hello,
> > 
> > I noticed these two warnings in clang.
> > 
> > It would be great if the 2nd patch could be checked by Sumit to make
> > sure that the return value wasn't ignored on purpose.
> 
> yes, I would prefer to ignore errors here. There might be various cases
> were we are not able to resolve a single SID but still can proceed with
> the others.
> 
> The first patch looks good, I'll run some tests and will give my results
> later.

ACK to the first patch.

bye,
Sumit

> 
> bye,
> Sumit
> 
> > 
> > Thanks,
> > 
> > Pavel Reichl
> > 
> _______________________________________________
> sssd-devel mailing list
> sssd-devel at lists.fedorahosted.org
> https://lists.fedorahosted.org/mailman/listinfo/sssd-devel



More information about the sssd-devel mailing list