[Freeipa-devel] [bind-dyndb-ldap PR#11][opened] Coverity: fix REVERSE_INULL for pevent->inst

tomaskrizek bind-dyndb-ldap-github-notification at redhat.com
Wed Mar 29 08:50:02 UTC 2017


   URL: https://github.com/freeipa/bind-dyndb-ldap/pull/11
Author: tomaskrizek
 Title: #11: Coverity: fix REVERSE_INULL for pevent->inst
Action: opened

PR body:
"""
With the DynDB API changes, the ldap instance is acquired
differently. Previously, obtaining the instance could fail when
LDAP was disconnecting, thus the NULL check was necessary in the
cleanup part.

Now, inst is obtained directly from the API. I'm not sure what is
the exact behaviour in edge cases such as LDAP disconnecting, so
I perform the NULL check a bit earlier, just to be safe.
"""

To pull the PR as Git branch:
git remote add ghbind-dyndb-ldap https://github.com/freeipa/bind-dyndb-ldap
git fetch ghbind-dyndb-ldap pull/11/head:pr11
git checkout pr11
-------------- next part --------------
A non-text attachment was scrubbed...
Name: bind-dyndb-ldap-pr-11.patch
Type: text/x-diff
Size: 3794 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20170329/a48d302f/attachment.bin>


More information about the Freeipa-devel mailing list