Attachment 12693 Details for Bug 13274 – patch take 2

[patch] patch take 2

crash_fix.diff (text/plain), 1.42 KB, created by Kusanagi Kouichi on 2007-11-22 22:11:49 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Kusanagi Kouichi

Created: 2007-11-22 22:11:49 UTC

Size: 1.42 KB

patch

obsolete

>diff --git a/src/radeon_video.c b/src/radeon_video.c
>index 26857a5..332f2cd 100644
>--- a/src/radeon_video.c
>+++ b/src/radeon_video.c
>@@ -2199,7 +2199,7 @@ RADEONCopyRGB24Data(
>   unsigned int w
> ){
>     CARD32 *dptr;
>-    CARD8 *sptr = 0;
>+    CARD8 *sptr;
>     int i,j;
>     RADEONInfoPtr info = RADEONPTR(pScrn);
> #ifdef XF86DRI
>@@ -2214,7 +2214,7 @@ RADEONCopyRGB24Data(
> 
> 	RADEONHostDataParams( pScrn, dst, dstPitch, 4, &dstPitchOff, &x, &y );
> 
>-	while ( (dptr = ( CARD32* )RADEONHostDataBlit( pScrn, 4, w, dstPitch,
>+	while ( (dptr = ( CARD32* )RADEONHostDataBlit( pScrn, 4, w, dstPitchOff,
> 						       &bufPitch, x, &y, &h,
> 						       &hpass )) )
> 	{
>@@ -2224,11 +2224,14 @@ RADEONCopyRGB24Data(
> 
> 		for ( i = 0 ; i < w; i++, sptr += 3 )
> 		{
>-		    *dptr++ = (sptr[0] << 24) | (sptr[1] << 16) | sptr[2];
>+#if X_BYTE_ORDER == X_BIG_ENDIAN
>+		    *dptr++ = (sptr[0] << 16) | (sptr[1] << 8) | sptr[2];
>+#else
>+		    *dptr++ = (sptr[2] << 16) | (sptr[1] << 8) | sptr[0];
>+#endif
> 		}
> 
>-		src += hpass * srcPitch;
>-		dptr += hpass * bufPitch;
>+		src += srcPitch;
> 	    }
> 	}
> 
>@@ -2251,8 +2254,11 @@ RADEONCopyRGB24Data(
> 	    sptr=src+j*srcPitch;
> 
> 	    for(i=w;i>0;i--){
>-	      dptr[0]=((sptr[0])<<24)|((sptr[1])<<16)|(sptr[2]);
>-	      dptr++;
>+#if X_BYTE_ORDER == X_BIG_ENDIAN
>+	      *dptr++=((sptr[0])<<16)|((sptr[1])<<8)|(sptr[2]);
>+#else
>+	      *dptr++=((sptr[2])<<16)|((sptr[1])<<8)|(sptr[0]);
>+#endif
> 	      sptr+=3;
> 	    }
> 	}

Actions: View | Splinter Review

Attachments on bug 13274: 12591 | 12693 | 12694

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.