Opened 14 years ago

Closed 14 years ago

#1594 closed defect (fixed)

Protect against bogus refresh rate reported by X

Reported by: gsstark@… Owned by: Isaac Richards
Priority: minor Milestone: unknown
Component: mythtv Version: 0.19
Severity: medium Keywords:
Cc: Ticket locked: no

Description

Mythtv uses XF86VidModeGetModeLine to calculate a refresh rate. But it appears the ATI driver is giving a bogus value that makes mythtv think the refresh rate is 434Khz. I could see how that would confuse mythtv.

I submitted an xorg bug report: https://bugs.freedesktop.org/show_bug.cgi?id=6421

But it seems like mythtv can easily protect itself here. It already defaults to 60hz if it can't get any refresh rate from X. I would say it should also fail over to the default if the refresh rate it gets from X is unreasonable.

I've attached an untested patch.

Attachments (1)

patch (1.0 KB) - added by anonymous 14 years ago.

Download all attachments as: .zip

Change History (2)

Changed 14 years ago by anonymous

Attachment: patch added

comment:1 Changed 14 years ago by Isaac Richards

Resolution: fixed
Status: newclosed

(In [9583]) Reject unreasonable refresh rates.

Closes #1594.

Note: See TracTickets for help on using tickets.