URL:
https://github.com/SSSD/sssd/pull/5375
Title: #5375: kcm: improve performance for large ccaches
alexey-tikhonov commented:
"""
Could you please explain clain "2)" in the "sss_ptr_hash: fix double free
for circular dependencies" patch?
Even taking into account `sss_ptr_hash_value` was created in the context of `payload`, I
still see no problem if custom cb deletes `payload` in this execution flow: `libtalloc`
will not enter `sss_ptr_hash_value_destructor` recursively (or what other reason for
"crash in talloc" you expect?)
I think existing test `test_sss_ptr_hash_with_free_cb` covers this. Or do I miss
something?
"""
See the full comment at
https://github.com/SSSD/sssd/pull/5375#issuecomment-733275701