| 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.