mirror of
https://github.com/X11Libre/xserver.git
synced 2026-10-22 01:20:26 +00:00
Allow for missing or disabled compat_output
When the compat output is missing (I don't think this is actually possible), or is disabled (and hence has no crtc), we would like to avoid dereferencing NULL pointers. This patch creates inline functions to extract the current compat output, crtc or associated RandR crtc structure, carefully checking for NULL pointers everywhere. Reviewed-by: Adam Jackson <ajax@redhat.com> Signed-off-by: Keith Packard <keithp@keithp.com>
This commit is contained in:
parent
fbbadca7e8
commit
de86a3a344
4 changed files with 46 additions and 17 deletions
|
|
@ -689,6 +689,32 @@ extern _X_EXPORT int xf86CrtcConfigPrivateIndex;
|
|||
|
||||
#define XF86_CRTC_CONFIG_PTR(p) ((xf86CrtcConfigPtr) ((p)->privates[xf86CrtcConfigPrivateIndex].ptr))
|
||||
|
||||
static _X_INLINE xf86OutputPtr
|
||||
xf86CompatOutput(ScrnInfoPtr pScrn)
|
||||
{
|
||||
xf86CrtcConfigPtr config = XF86_CRTC_CONFIG_PTR(pScrn);
|
||||
return config->output[config->compat_output];
|
||||
}
|
||||
|
||||
static _X_INLINE xf86CrtcPtr
|
||||
xf86CompatCrtc(ScrnInfoPtr pScrn)
|
||||
{
|
||||
xf86OutputPtr compat_output = xf86CompatOutput(pScrn);
|
||||
if (!compat_output)
|
||||
return NULL;
|
||||
return compat_output->crtc;
|
||||
}
|
||||
|
||||
static _X_INLINE RRCrtcPtr
|
||||
xf86CompatRRCrtc(ScrnInfoPtr pScrn)
|
||||
{
|
||||
xf86CrtcPtr compat_crtc = xf86CompatCrtc(pScrn);
|
||||
if (!compat_crtc)
|
||||
return NULL;
|
||||
return compat_crtc->randr_crtc;
|
||||
}
|
||||
|
||||
|
||||
/*
|
||||
* Initialize xf86CrtcConfig structure
|
||||
*/
|
||||
|
|
|
|||
Loading…
Reference in a new issue