URL: https://github.com/freeipa/bind-dyndb-ldap/pull/11
Title: #11: Coverity: fix REVERSE_INULL for pevent->inst
tomaskrizek commented:
"""
master:
- 13b185182aeb48562cf63251b84bcf910b57a0fc
"""
See the full comment at
https://github.com/freeipa/bind-dyndb-ldap/pull/11#issuecomment-292221521
URL: https://github.com/freeipa/bind-dyndb-ldap/pull/11
Title: #11: Coverity: fix REVERSE_INULL for pevent->inst
tomaskrizek commented:
"""
Thanks for the review, Petr!
I added the check to `syncrepl_update` as well.
"""
See the full comment at
URL: https://github.com/freeipa/bind-dyndb-ldap/pull/11
Title: #11: Coverity: fix REVERSE_INULL for pevent->inst
pemensik commented:
"""
Hi Tomáš,
I did not find any place which could result with inst == NULL.
ldap_sync_prepare contains proper REQUIRE(inst != null) and all other function
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