Re: [Freeipa-devel] [PATCH] 1104 move NULL check

2013-08-29 Thread Petr Viktorin

On 08/29/2013 02:57 PM, Tomas Babej wrote:

On 08/28/2013 04:40 PM, Rob Crittenden wrote:

Move a NULL check in the lockout plugin to address something Coverity
found. There is no risk of a crash here but it there is also no point
calling something if we know the values are NULL.

rob


___
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel


ACK



Pushed to
master: 3940a574f95750e6bd02dcc30953db0fe9146f0a
ipa-3-3: 0688827f3c287c1ec354f90e4748b0c23c4c9883


--
PetrĀ³

___
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel


Re: [Freeipa-devel] [PATCH] 1104 move NULL check

2013-08-29 Thread Tomas Babej

On 08/28/2013 04:40 PM, Rob Crittenden wrote:
Move a NULL check in the lockout plugin to address something Coverity 
found. There is no risk of a crash here but it there is also no point 
calling something if we know the values are NULL.


rob


___
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel


ACK

--
Tomas Babej
Associate Software Engeneer | Red Hat | Identity Management
RHCE | Brno Site | IRC: tbabej | freeipa.org

___
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel