URL:
https://github.com/SSSD/sssd/pull/883
Title: #883: Minor fixes to util/sss_krb5
alexey-tikhonov commented:
"""
I'm kind of confused that you think that, since we pretty clearly
disagree: you claim that you've found a double-free, and I'm telling you that
free(NULL) is fine and normal.
`free(NULL)` is fine and normal because man page says so explicitly.
`krb5_free_keytab_entry_contents(structure_that_was_memset(0))` is not until docs
explicitly say other way.
Lets put it other way: do you have any reasonable objection against patch?
I can edit commit message replacing "fixed error" with "put sanity
guard" if you want.
"""
See the full comment at
https://github.com/SSSD/sssd/pull/883#issuecomment-533326689