Bug 6150

Summary: Check in xf86InitFBManager() is always false
Product: xorg Reporter: Julio Merino <jmmv>
Component: Server/GeneralAssignee: Xorg Project Team <xorg-team>
Status: RESOLVED FIXED QA Contact:
Severity: minor    
Priority: high Keywords: patch
Version: git   
Hardware: All   
OS: All   
i915 platform: i915 features:
Description Flags
Sample patch. none

Description Julio Merino 2006-03-06 00:27:37 UTC
The xf86InitFBManager function in the  
xc/programs/Xserver/hw/xfree86/common/xf86fbman.c file contains a sanity check  
for input parameters that is always false.  This is found in the following  
    if (FullBox->x2 < FullBox->x2) return FALSE;  
As you can see, FullBox->x2 will always be equal to itself, so this check will  
never evaluate to true and do the 'return FALSE'. 
I think that the second FullBox->x2 in the conditional should be FullBox->x1 
similar to what the previous line in the file does for the Y coordinate.  See 
the attached patch.
Comment 1 Julio Merino 2006-03-06 00:28:17 UTC
Created attachment 4828 [details] [review]
Sample patch.
Comment 2 Eric Anholt 2006-03-08 14:32:14 UTC
Committed.  Thanks!

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.