[SSSD] [PATCH] explicit null dereferenced in sss_nss_mc_get_record()

Stephen Gallagher sgallagh at redhat.com
Mon Jan 7 21:22:05 UTC 2013


On Mon 07 Jan 2013 11:46:49 AM EST, Jakub Hrozek wrote:
> On Mon, Jan 07, 2013 at 10:05:53AM -0500, Simo Sorce wrote:
>> On Mon, 2013-01-07 at 15:39 +0100, Pavel Březina wrote:
>>> Thanks, new patches are attached.
>>>
>>> I also changed logic of MC_CHECK_RECORD_LENGTH() so it returns true
>>> when the length is valid.
>>
>> ACK to all three.
>>
>> Simo.
>
> Pushed to master and sssd-1-9



Too late to stop the push, but we could be leaking memory for copy_rec 
if we fail MC_CHECK_RECORD_LENGTH on a pass through the loop beyond the 
first. Coverity scan discovered it.



More information about the sssd-devel mailing list