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

2017-04-06 Thread tomaskrizek
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

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

2017-04-03 Thread tomaskrizek
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

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

2017-03-30 Thread pemensik
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

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

2017-03-29 Thread tomaskrizek
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