On 01/31/11 04:02 AM, Erkki Seppälä wrote: > From: Ander Conselvan de Oliveira <ander.conselvan-de-olive...@nokia.com> > > xcms/LRGB: Fix potential resource leak. > > Reviewed-by: Erkki Seppälä <erkki.sepp...@vincit.fi> > Signed-off-by: Ander Conselvan de Oliveira > <ander.conselvan-de-olive...@nokia.com> > Signed-off-by: Erkki Seppälä <erkki.sepp...@vincit.fi> > --- > src/xcms/LRGB.c | 2 ++ > 1 files changed, 2 insertions(+), 0 deletions(-) > > diff --git a/src/xcms/LRGB.c b/src/xcms/LRGB.c > index d9168bd..c1606be 100644 > --- a/src/xcms/LRGB.c > +++ b/src/xcms/LRGB.c > @@ -686,6 +686,7 @@ LINEAR_RGB_InitSCCData( > /* Red Intensity Table */ > if (!(pScreenData->pRedTbl = (IntensityTbl *) > Xcalloc (1, sizeof(IntensityTbl)))) { > + XFree ((char * ) property_return); > goto FreeSCCData; > } > if (_XcmsGetTableType0(pScreenData->pRedTbl, format_return, &pChar, > @@ -723,6 +724,7 @@ LINEAR_RGB_InitSCCData( > /* Red Intensity Table */ > if (!(pScreenData->pRedTbl = (IntensityTbl *) > Xcalloc (1, sizeof(IntensityTbl)))) { > + XFree ((char * ) property_return); > goto FreeSCCData; > } > if (_XcmsGetTableType1(pScreenData->pRedTbl, format_return, &pChar,
Reviewed-by: Alan Coopersmith <alan.coopersm...@oracle.com> -- -Alan Coopersmith- alan.coopersm...@oracle.com Oracle Solaris Platform Engineering: X Window System _______________________________________________ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel