fdo-mirrors/xserver
1
0
Fork 0
mirror of https://gitlab.freedesktop.org/xorg/xserver.git synced 2026-05-15 18:28:06 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions

XQuartz: GL: setVisualConfigs() copied the behavior of indirect.c which is wrong, with regard to the handling of xcalloc failure.

Browse source 
Use ErrorF for an error message after an xcalloc failure, and return instead of falling through to GlxSetVisualConfigs, or abort()ing at the test branch.
(cherry picked from commit 1056700971)
This commit is contained in:
George Staplin 2008-10-07 12:12:38 -06:00 • committed by Jeremy Huddleston
parent f1fec6f323
commit 8033279294
1 changed files with 57 additions and 47 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

104
hw/xquartz/GL/visualConfigs.c
View file

@ -86,65 +86,75 @@ void setVisualConfigs(void) {
2 iterations for accum (on and off (with an accum color size of 16)).
*/
numConfigs = ((enable_stereo && caps->stereo) ? 2 : 1) * 2 *
(caps->aux_buffers ? 2 : 1) * (caps->buffers) * 2 * 2;
visualConfigs = xcalloc(sizeof(*visualConfigs), numConfigs);
if(NULL == visualConfigs) {
ErrorF("xcalloc failure when allocating visualConfigs\n");
return;
}
visualPrivates = xcalloc(sizeof(void *), numConfigs);
if (NULL != visualConfigs) {
i = 0; /* current buffer */
for (stereo = 0; stereo < ((enable_stereo && caps->stereo) ? 2 : 1); ++stereo) {
for (depth = 0; depth < 2; ++depth) {
for (aux = 0; aux < (caps->aux_buffers ? 2 : 1); ++aux) {
for (buffers = 0; buffers < caps->buffers; ++buffers) {
for (stencil = 0; stencil < 2; ++stencil) {
for (accum = 0; accum < 2; ++accum) {
visualConfigs[i].vid = -1;
visualConfigs[i].class = -1;
visualConfigs[i].rgba = TRUE;
visualConfigs[i].redSize = -1;
visualConfigs[i].greenSize = -1;
visualConfigs[i].blueSize = -1;
visualConfigs[i].redMask = -1;
visualConfigs[i].greenMask = -1;
visualConfigs[i].blueMask = -1;
visualConfigs[i].alphaMask = 0;
if (accum) {
visualConfigs[i].accumRedSize = 16;
visualConfigs[i].accumGreenSize = 16;
visualConfigs[i].accumBlueSize = 16;
visualConfigs[i].accumAlphaSize = 16;
} else {
visualConfigs[i].accumRedSize = 0;
visualConfigs[i].accumGreenSize = 0;
visualConfigs[i].accumBlueSize = 0;
visualConfigs[i].accumAlphaSize = 0;
}
visualConfigs[i].doubleBuffer = buffers ? TRUE : FALSE;
visualConfigs[i].stereo = stereo ? TRUE : FALSE;
visualConfigs[i].bufferSize = -1;
visualConfigs[i].depthSize = depth ? 24 : 0;
visualConfigs[i].stencilSize = stencil ? 8 : 0;
visualConfigs[i].auxBuffers = aux ? caps->aux_buffers : 0;
visualConfigs[i].level = 0;
visualConfigs[i].visualRating = GLX_NONE_EXT;
visualConfigs[i].transparentPixel = 0;
visualConfigs[i].transparentRed = 0;
visualConfigs[i].transparentGreen = 0;
visualConfigs[i].transparentBlue = 0;
visualConfigs[i].transparentAlpha = 0;
visualConfigs[i].transparentIndex = 0;
++i;
if(NULL == visualPrivates) {
ErrorF("xcalloc failure when allocating visualPrivates");
xfree(visualConfigs);
return;
}
i = 0; /* current buffer */
for (stereo = 0; stereo < ((enable_stereo && caps->stereo) ? 2 : 1); ++stereo) {
for (depth = 0; depth < 2; ++depth) {
for (aux = 0; aux < (caps->aux_buffers ? 2 : 1); ++aux) {
for (buffers = 0; buffers < caps->buffers; ++buffers) {
for (stencil = 0; stencil < 2; ++stencil) {
for (accum = 0; accum < 2; ++accum) {
visualConfigs[i].vid = -1;
visualConfigs[i].class = -1;
visualConfigs[i].rgba = TRUE;
visualConfigs[i].redSize = -1;
visualConfigs[i].greenSize = -1;
visualConfigs[i].blueSize = -1;
visualConfigs[i].redMask = -1;
visualConfigs[i].greenMask = -1;
visualConfigs[i].blueMask = -1;
visualConfigs[i].alphaMask = 0;
if (accum) {
visualConfigs[i].accumRedSize = 16;
visualConfigs[i].accumGreenSize = 16;
visualConfigs[i].accumBlueSize = 16;
visualConfigs[i].accumAlphaSize = 16;
} else {
visualConfigs[i].accumRedSize = 0;
visualConfigs[i].accumGreenSize = 0;
visualConfigs[i].accumBlueSize = 0;
visualConfigs[i].accumAlphaSize = 0;
}
visualConfigs[i].doubleBuffer = buffers ? TRUE : FALSE;
visualConfigs[i].stereo = stereo ? TRUE : FALSE;
visualConfigs[i].bufferSize = -1;
visualConfigs[i].depthSize = depth ? 24 : 0;
visualConfigs[i].stencilSize = stencil ? 8 : 0;
visualConfigs[i].auxBuffers = aux ? caps->aux_buffers : 0;
visualConfigs[i].level = 0;
visualConfigs[i].visualRating = GLX_NONE_EXT;
visualConfigs[i].transparentPixel = 0;
visualConfigs[i].transparentRed = 0;
visualConfigs[i].transparentGreen = 0;
visualConfigs[i].transparentBlue = 0;
visualConfigs[i].transparentAlpha = 0;
visualConfigs[i].transparentIndex = 0;
++i;
}
}
}
}
}
}
}
if (i != numConfigs) {