diff options
| author | Alan Coopersmith <alan.coopersmith@sun.com> | 2009-12-11 00:46:22 -0800 |
|---|---|---|
| committer | Alan Coopersmith <alan.coopersmith@sun.com> | 2009-12-18 16:51:45 -0800 |
| commit | eb750f8b5e14751d4c40b50499baec5d2ba79db9 (patch) | |
| tree | 48791902f8ebe6f6422787285accb13d44e25b8a /Xext/panoramiX.c | |
| parent | 1df4bd6011e110dcf0649b15bfffd4ab9e6961d6 (diff) | |
Check for failures from CreateNewResourceType
Make sure to check return value before setting bitmask flags.
For most calls, just fails to init the extension. Since Xinput
already calls FatalError() on initialization failure, so does
failure to allocate Xinput's resource type.
Signed-off-by: Alan Coopersmith <alan.coopersmith@sun.com>
Reviewed-by: Keith Packard <keithp@keithp.com>
Diffstat (limited to 'Xext/panoramiX.c')
| -rw-r--r-- | Xext/panoramiX.c | 16 |
1 files changed, 10 insertions, 6 deletions
diff --git a/Xext/panoramiX.c b/Xext/panoramiX.c index 4b7b07e1d..f48bd36a6 100644 --- a/Xext/panoramiX.c +++ b/Xext/panoramiX.c @@ -503,15 +503,19 @@ void PanoramiXExtensionInit(int argc, char *argv[]) } XRC_DRAWABLE = CreateNewResourceClass(); - XRT_WINDOW = CreateNewResourceType(XineramaDeleteResource) | - XRC_DRAWABLE; - XRT_PIXMAP = CreateNewResourceType(XineramaDeleteResource) | - XRC_DRAWABLE; + XRT_WINDOW = CreateNewResourceType(XineramaDeleteResource); + if (XRT_WINDOW) + XRT_WINDOW |= XRC_DRAWABLE; + XRT_PIXMAP = CreateNewResourceType(XineramaDeleteResource); + if (XRT_PIXMAP) + XRT_PIXMAP |= XRC_DRAWABLE; XRT_GC = CreateNewResourceType(XineramaDeleteResource); XRT_COLORMAP = CreateNewResourceType(XineramaDeleteResource); - panoramiXGeneration = serverGeneration; - success = TRUE; + if (XRT_WINDOW && XRT_PIXMAP && XRT_GC && XRT_COLORMAP) { + panoramiXGeneration = serverGeneration; + success = TRUE; + } } if (!success) { |