Summary: | Signal 11 at startup with Matrox G450 | ||||||
---|---|---|---|---|---|---|---|
Product: | xorg | Reporter: | Tony Houghton <h> | ||||
Component: | Driver/mga | Assignee: | Adam Jackson <ajax> | ||||
Status: | RESOLVED FIXED | QA Contact: | |||||
Severity: | critical | ||||||
Priority: | high | CC: | mharris, michal.jnn | ||||
Version: | 6.8.2 | ||||||
Hardware: | x86 (IA32) | ||||||
OS: | Linux (All) | ||||||
Whiteboard: | |||||||
i915 platform: | i915 features: | ||||||
Bug Depends on: | |||||||
Bug Blocks: | 1690 | ||||||
Attachments: |
|
Description
Tony Houghton
2005-10-24 07:32:45 UTC
line 217 contains the following gem: VDIF_Block = VDIFRead(scrnIndex, pBus, EDID1_LEN * (tmp->no_sections + 1)); we do not check that tmp is non-null. we get it from xf86InterpretEDID, which can definitely return null, so that's pretty awesome. should probably wrap that stanza in if (tmp). Created attachment 3742 [details] [review] Patch to fix this bug in 6.8.2 I added a check for tmp being NULL as suggested, and the server now starts successfully. This is a patch against the Debian package version 6.8.2.dfsg.1-1. applied, 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.