frederik
/
xserver
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

xnest: protect from memory allocation failure 

Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
This commit is contained in:
Enrico Weigelt, metux IT consult 2025-02-24 13:57:51 +01:00 committed by Enrico Weigelt, metux IT consult .
parent 6a1ec27678
commit 39b2354bbc
5 changed files with 24 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
hw/xnest/Color.c
View File

@ -64,6 +64,8 @@ xnestCreateColormap(ColormapPtr pCmap)
switch (pVisual->class) {
case StaticGray: /* read only */
colors = calloc(ncolors, sizeof(XColor));
if (!colors)
return FALSE;
for (i = 0; i < ncolors; i++)
colors[i].pixel = i;
XQueryColors(xnestDisplay, xnestColormap(pCmap), colors, ncolors);
@ -77,6 +79,8 @@ xnestCreateColormap(ColormapPtr pCmap)
case StaticColor: /* read only */
colors = calloc(ncolors, sizeof(XColor));
if (!colors)
return FALSE;
for (i = 0; i < ncolors; i++)
colors[i].pixel = i;
XQueryColors(xnestDisplay, xnestColormap(pCmap), colors, ncolors);
@ -90,6 +94,8 @@ xnestCreateColormap(ColormapPtr pCmap)
case TrueColor: /* read only */
colors = calloc(ncolors, sizeof(XColor));
if (!colors)
return FALSE;
red = green = blue = 0L;
redInc = lowbit(pVisual->redMask);
greenInc = lowbit(pVisual->greenMask);
@ -196,11 +202,17 @@ xnestSetInstalledColormapWindows(ScreenPtr pScreen)
int numWindows;
icws.cmapIDs = calloc(pScreen->maxInstalledCmaps, sizeof(Colormap));
if (!icws.cmapIDs)
return;
icws.numCmapIDs = xnestListInstalledColormaps(pScreen, icws.cmapIDs);
icws.numWindows = 0;
WalkTree(pScreen, xnestCountInstalledColormapWindows, (void *) &icws);
if (icws.numWindows) {
icws.windows = calloc(icws.numWindows + 1, sizeof(Window));
if (!icws.windows) {
free(icws.cmapIDs);
return;
}
icws.index = 0;
WalkTree(pScreen, xnestGetInstalledColormapWindows, (void *) &icws);
icws.windows[icws.numWindows] = xnestDefaultWindows[pScreen->myNum];
@ -220,6 +232,8 @@ xnestSetInstalledColormapWindows(ScreenPtr pScreen)
{
int i;
Window64 *windows = calloc(numWindows, sizeof(Window64));
if (!windows)
return;
for (i = 0; i < numWindows; ++i)
windows[i] = icws.windows[i];
@ -392,6 +406,8 @@ xnestStoreColors(ColormapPtr pCmap, int nColors, xColorItem * pColors)
{
int i;
XColor *pColors64 = calloc(nColors, sizeof(XColor));
if (!pColors64)
return;
for (i = 0; i < nColors; ++i) {
pColors64[i].pixel = pColors[i].pixel;

2
hw/xnest/Display.c
View File

@ -118,6 +118,8 @@ xnestOpenDisplay(int argc, char *argv[])
xnestNumDefaultColormaps = xnestNumVisuals;
xnestDefaultColormaps = calloc(xnestNumDefaultColormaps, sizeof(Colormap));
if (!xnestDefaultColormaps)
return;
for (i = 0; i < xnestNumDefaultColormaps; i++)
xnestDefaultColormaps[i] = XCreateColormap(xnestDisplay,
DefaultRootWindow

2
hw/xnest/GC.c
View File

@ -204,6 +204,8 @@ xnestChangeClip(GCPtr pGC, int type, void *pValue, int nRects)
case CT_REGION:
nRects = RegionNumRects((RegionPtr) pValue);
pRects = calloc(nRects, sizeof(*pRects));
if (!pRects)
break;
pBox = RegionRects((RegionPtr) pValue);
for (i = nRects; i-- > 0;) {
pRects[i].x = pBox[i].x1;

3
hw/xnest/Keyboard.c
View File

@ -119,7 +119,8 @@ xnestKeyboardProc(DeviceIntPtr pDev, int onoff)
max_keycode - min_keycode + 1,
&mapWidth);
len = (max_keycode - min_keycode + 1) * mapWidth;
keymap = calloc(len, sizeof(KeySym));
if (!(keymap = calloc(len, sizeof(KeySym))))
goto XkbError;
for (i = 0; i < len; ++i)
keymap[i] = keymap64[i];
XFree(keymap64);

2
hw/xnest/Screen.c
View File

@ -165,6 +165,8 @@ xnestOpenScreen(ScreenPtr pScreen, int argc, char *argv[])
return FALSE;
visuals = calloc(xnestNumVisuals, sizeof(VisualRec));
if (!visuals)
return FALSE;
numVisuals = 0;
depths = (DepthPtr) malloc(MAXDEPTH * sizeof(DepthRec));