xf86-video-intel: src/intel_video.c src/sna/sna_video_overlay.c

[Dave Airlie]

 src/intel_video.c           |    2 +-
 src/sna/sna_video_overlay.c |    2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

New commits:
commit b580abdfa68108f3e63ee1f897b6ea83b9c47935
Author: Dave Airlie <airlied at gmail.com>
Date:   Sun Aug 12 20:52:12 2012 +1000

    intel: fix video xvPipe range check
    
    Pointed out by Russell King on irc.
    
    Signed-off-by: Dave Airlie <airlied at redhat.com>

diff --git a/src/intel_video.c b/src/intel_video.c
index c8a9a92..0be72e2 100644
--- a/src/intel_video.c
+++ b/src/intel_video.c
@@ -675,7 +675,7 @@ I830SetPortAttributeOverlay(ScrnInfoPtr scrn,
 		adaptor_priv->saturation = value;
 	} else if (attribute == xvPipe) {
 		xf86CrtcConfigPtr xf86_config = XF86_CRTC_CONFIG_PTR(scrn);
-		if ((value < -1) || (value > xf86_config->num_crtc))
+		if ((value < -1) || (value >= xf86_config->num_crtc))
 			return BadValue;
 		if (value < 0)
 			adaptor_priv->desired_crtc = NULL;
diff --git a/src/sna/sna_video_overlay.c b/src/sna/sna_video_overlay.c
index 068f234..babdfc6 100644
--- a/src/sna/sna_video_overlay.c
+++ b/src/sna/sna_video_overlay.c
@@ -194,7 +194,7 @@ sna_video_overlay_set_port_attribute(ScrnInfoPtr scrn,
 		video->saturation = value;
 	} else if (attribute == xvPipe) {
 		xf86CrtcConfigPtr xf86_config = XF86_CRTC_CONFIG_PTR(scrn);
-		if ((value < -1) || (value > xf86_config->num_crtc))
+		if ((value < -1) || (value >= xf86_config->num_crtc))
 			return BadValue;
 		if (value < 0)
 			video->desired_crtc = NULL;