On Thu, 3 Apr 2003, Alexandr Andreev wrote:
> I guess there is a bug in the fbdevHWSetVideoModes() function.
> void
> fbdevHWSetVideoModes(ScrnInfoPtr pScrn)
> {
> ...
> DisplayModePtr mode,this,last = NULL;
>
> ...
> if (NULL == pScrn->modes) {
> ...
> } else {
> this = xnfalloc(sizeof(DisplayModeRec));
> memcpy(this,mode,sizeof(DisplayModeRec));
> this->next = pScrn->modes;
> this->prev = last;
> last->next = this; /* Oops... this is not good. 'last'
> is NULL at the first time. */
> pScrn->modes->prev = this;
> }
> last = this;
> ...
> }
> I have XFree86-4.2.0 but nothing has changed in this function in 4.3.0.
> I think this patch may help.
I have committed this fix. Thanks for reporting the problem.
Marc.
+----------------------------------+-----------------------------------+
| Marc Aurele La France | work: 1-780-492-9310 |
| Computing and Network Services | fax: 1-780-492-1729 |
| 352 General Services Building | email: [EMAIL PROTECTED] |
| University of Alberta +-----------------------------------+
| Edmonton, Alberta | |
| T6G 2H1 | Standard disclaimers apply |
| CANADA | |
+----------------------------------+-----------------------------------+
XFree86 Core Team member. ATI driver and X server internals.
_______________________________________________
Devel mailing list
[EMAIL PROTECTED]
http://XFree86.Org/mailman/listinfo/devel
