[Freeipa-devel] [PATCH] 0024 memory leak in ipapwd plugin
Alexander Bokovoy
abokovoy at redhat.com
Thu Aug 11 14:39:03 UTC 2016
On Thu, 11 Aug 2016, thierry bordaz wrote:
>>>+ /* rc should always be 0 (else slapi_sdn_new_dn_byref
>>>should have sigsev)
>>>+ * but if we end in rc==LDAP_OPERATIONS_ERROR be sure to
>>>stop here
>>>+ * because ret is not significant */
>>A short note here. You talk about slapi_sdn_new_dn_byref() but your
>>patch replaces that with slapi_sdn_new_dn_byval(). Does the comment
>>still apply?
>>
>>>+ if (rc != 0) {
>>>+ LOG_OOM();
>>>+ goto free_and_return;
>>>+ }
>>>+
>>> if (ret == 0) {
>>> Slapi_Value *cpw[2] = { NULL, NULL };
>>> Slapi_Value *pw;
>>>--
>>>2.7.4
>>>
>>
>>
>Good catch Alexander. Yes the comment contained a wrong cut/paste
ACK.
--
/ Alexander Bokovoy
More information about the Freeipa-devel
mailing list