[SSSD] [PATCH] Possible use of NULL pointer in function, sss_mmap_cache_init.
Michal Židek
mzidek at redhat.com
Thu Sep 27 09:19:41 UTC 2012
On 09/26/2012 07:27 PM, Stephen Gallagher wrote:
> On Wed 26 Sep 2012 12:52:07 PM EDT, Michal Židek wrote:
>> NULL pointer was passed to unlink() if memory allocation
>> for mc_ctx->file failed.
>>
>
> Nack. We should be checking for the allocation failure and handling that
> earlier.
Actually, this is part of the clean-up process, when the allocation was
unsuccessful, but I've checked the code flow, that clang's static
analysis reported as problematic again and it is impossible to happen,
so the patch is not needed.
Michal
More information about the sssd-devel
mailing list