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

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


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

tomaskrizek commented:
"""
@pemensik Hi, could you take a quick look at this change?

I ran coverity and the issues were fixed.

It might also be possible to remove the REQUIRE, but since I'm not sure whether `inst` is always non null in the new dyndb workflow, I added the check just to be sure.
"""

See the full comment at https://github.com/freeipa/bind-dyndb-ldap/pull/11#issuecomment-290026409


More information about the Freeipa-devel mailing list