[ros-diffs] [tkreuzer] 32461: don't free the memory for the palette entries in IntSetPalette. Fixes bug 3072 See issue #3072 for more details.

[tkreuzer at svn.reactos.org]

Author: tkreuzer
Date: Sun Feb 24 06:12:38 2008
New Revision: 32461

URL: http://svn.reactos.org/svn/reactos?rev=32461&view=rev
Log:
don't free the memory for the palette entries in IntSetPalette. Fixes bug 3072
See issue #3072 for more details.

Modified:
    trunk/reactos/drivers/video/displays/framebuf/palette.c

Modified: trunk/reactos/drivers/video/displays/framebuf/palette.c
URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/video/displays/framebuf/palette.c?rev=32461&r1=32460&r2=32461&view=diff
==============================================================================
--- trunk/reactos/drivers/video/displays/framebuf/palette.c (original)
+++ trunk/reactos/drivers/video/displays/framebuf/palette.c Sun Feb 24 06:12:38 2008
@@ -121,7 +121,6 @@
    pClut->FirstEntry = iStart;
    pClut->NumEntries = cColors;
    memcpy(&pClut->LookupTable[0].RgbLong, ppalent, sizeof(ULONG) * cColors);
-   EngFreeMem(ppalent);
 
    if (((PPDEV)dhpdev)->PaletteShift)
    {
@@ -193,5 +192,7 @@
       return FALSE;
    }
 
-   return IntSetPalette(dhpdev, PaletteEntries, iStart, cColors);
+   BOOL bRet = IntSetPalette(dhpdev, PaletteEntries, iStart, cColors);
+   EngFreeMem(PaletteEntries);
+   return bRet;
 }