Stephen Gallagher wrote:
On 07/02/2010 09:37 AM, Dmitri Pal wrote:
Nack
You are leaking entry in success scenario. I suggest a little bit cleaner approach:
while((ret = krb5_kt_next_entry(context, keytab, &entry, &cursor))== 0){ krb5_unparse_name(context, entry.principal, &principal); krb5_free_keytab_entry_contents(context, &entry); if (strcmp(full_princ, principal) == 0) found = true; free(principal); if (found) break; }
Good catch. Fixed.
And I think that if you have explicit syslog messages in case of failures it makes sense to have one also in the case krb5_kt_end_seq_get fails.
Sure. It should hopefully never happen, but added nonetheless.
Thanks for the review! New patch attached.
Visual ack.
-------------------------
_______________________________________________ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://fedorahosted.org/mailman/listinfo/sssd-devel