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

Jakub Hrozek jhrozek at redhat.com
Mon Jan 7 21:47:45 UTC 2013


On Mon, Jan 07, 2013 at 04:22:05PM -0500, Stephen Gallagher wrote:
> 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.

Thank you, Stephen:
https://fedorahosted.org/sssd/ticket/1748



More information about the sssd-devel mailing list