[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

X Strike Force X.Org X11 SVN commit: r912 - in trunk/debian: . patches patches/general



Author: ender
Date: 2005-11-30 12:52:43 -0500 (Wed, 30 Nov 2005)
New Revision: 912

Added:
   trunk/debian/patches/general/033_radeon_fix_for_no_accel.diff
Modified:
   trunk/debian/changelog
   trunk/debian/patches/series
Log:
* general/033_radeon_fix_for_no_accel.diff: A bug into one of ati driver's
  sources (radeon_video.c) causes Xorg server to crash when using it with
  option "NoAccel", due to erroneous dereferencing of uninitialized pointer.
  Thanks, Riccardo Brigo (closes: #341351).

Also fixed a typo in the changelog.


Modified: trunk/debian/changelog
===================================================================
--- trunk/debian/changelog	2005-11-29 16:49:33 UTC (rev 911)
+++ trunk/debian/changelog	2005-11-30 17:52:43 UTC (rev 912)
@@ -1,8 +1,12 @@
 xorg-x11 (6.8.2.dfsg.1-11+SVN) unstable; urgency=low
 
-  * 
+  [ David Martínez Moreno ]
+  * general/033_radeon_fix_for_no_accel.diff: A bug into one of ati driver's
+    sources (radeon_video.c) causes Xorg server to crash when using it with
+    option "NoAccel", due to erroneous dereferencing of uninitialized pointer.
+    Thanks, Riccardo Brigo (closes: #341351).
 
- -- David Nusinow <dnusinow@debian.org>  Tue, 29 Nov 2005 11:48:51 -0500
+ -- David Martínez Moreno <ender@debian.org>  Wed, 30 Nov 2005 18:49:26 +0100
 
 xorg-x11 (6.8.2.dfsg.1-11) unstable; urgency=low
 
@@ -37,7 +41,7 @@
 xorg-x11 (6.8.2.dfsg.1-10) unstable; urgency=low
 
   * Really fix x11-common.preinst "eq" test by using "=" instead.
-    (Closes: #334345
+    (Closes: #334345).
   * Add debian/992_debian_allow_build_from_svn.diff to ignore .svn directories
     rather than CVS ones, thereby letting people build straight from our SVN
     without failing MANIFEST check. Thanks Will Dyson and Frans Pop.

Added: trunk/debian/patches/general/033_radeon_fix_for_no_accel.diff
===================================================================
--- trunk/debian/patches/general/033_radeon_fix_for_no_accel.diff	2005-11-29 16:49:33 UTC (rev 911)
+++ trunk/debian/patches/general/033_radeon_fix_for_no_accel.diff	2005-11-30 17:52:43 UTC (rev 912)
@@ -0,0 +1,21 @@
+$Id:$
+
+A bug into one of ati driver's sources (radeon_video.c) causes Xorg server to
+crash when using it with option "NoAccel", due to erroneous dereferencing of
+uninitialized pointer.
+
+Fixes #341351. Patch by Riccardo Brigo.
+
+Index: xc/programs/Xserver/hw/xfree86/drivers/ati/radeon_video.c
+===================================================================
+--- xc/programs/Xserver/hw/xfree86/drivers/ati/radeon_video.c.orig	2005-07-19 11:30:50.583409940 +0200
++++ xc/programs/Xserver/hw/xfree86/drivers/ati/radeon_video.c	2005-11-30 18:43:41.326013490 +0100
+@@ -993,7 +993,7 @@
+     RADEONPortPrivPtr	pPriv = (RADEONPortPrivPtr)data;
+     Bool		setTransform = FALSE;
+ 
+-    info->accel->Sync(pScrn);
++    if (info->accelOn) info->accel->Sync(pScrn);
+ 
+ #define RTFSaturation(a)   (1.0 + ((a)*1.0)/1000.0)
+ #define RTFBrightness(a)   (((a)*1.0)/2000.0)


Property changes on: trunk/debian/patches/general/033_radeon_fix_for_no_accel.diff
___________________________________________________________________
Name: svn:keywords
   + Id

Modified: trunk/debian/patches/series
===================================================================
--- trunk/debian/patches/series	2005-11-29 16:49:33 UTC (rev 911)
+++ trunk/debian/patches/series	2005-11-30 17:52:43 UTC (rev 912)
@@ -51,6 +51,7 @@
 general/029_only_open_proc_bus_pci_devices_once.diff -p0
 general/031_disable_libXft1.diff
 general/032_fix_matrox_display.diff
+general/033_radeon_fix_for_no_accel.diff -p0
 general/035_tdfx_disable_dri_on_16mb_with_highres.diff -p0
 general/040_extend_netmouse_support.diff -p0
 general/041_make_xcursor_icondir_configurable.diff -p0



Reply to: