On 04/03/2013 10:39 PM, Dmitri Pal wrote:
On 04/03/2013 07:36 AM, Ondrej Kos wrote:
Hi,

Issue found by coverity, the code itself was inaccessibile.

https://fedorahosted.org/sssd/ticket/1857

Patch is attached

Ondra


_______________________________________________
sssd-devel mailing list
sssd-devel@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/sssd-devel


Did the unit test work work fine with this patch?
Should be.
I read the code but did nit run it. Seems like a visual ack.

Hi Dmitri,

Yes, the tests are running smoothly. I thoroughly investigated and tested this part of code, and this condition would definitely take false branch every time.


--
Thank you,
Dmitri Pal

Sr. Engineering Manager for IdM portfolio
Red Hat Inc.


-------------------------------
Looking to carve out IT costs?
www.redhat.com/carveoutcosts/




_______________________________________________
sssd-devel mailing list
sssd-devel@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/sssd-devel



--
Ondrej Kos
Associate Software Engineer
Identity Management
Red Hat Czech
_______________________________________________
sssd-devel mailing list
sssd-devel@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/sssd-devel

Reply via email to