[coreboot - Bug #475] (Resolved) mainboard_vbt_filename() in src/mainboard/lenovo/x200/blc.c does not work as intended

Martin Roth <[email protected]>
Newsgroups gmane.linux.bios
Message-ID <[email protected]>
Issue #475 has been updated by Martin Roth.

Status changed from New to Resolved

----------------------------------------
Bug #475: mainboard_vbt_filename() in src/mainboard/lenovo/x200/blc.c does not work as intended
https://ticket.coreboot.org/issues/475#change-1698

* Author: Bill XIE
* Status: Resolved
* Priority: Normal
* Assignee: Bill XIE
* Category: board support
* Target version: master
* Start date: 2023-03-30
* Affected versions: 4.15, 4.16, 4.17, 4.18, 4.19, master
* Related links: This is going to be fixed with https://review.coreboot.org/c/coreboot/+/74181 .
* Affected hardware: x200
----------------------------------------
Date back to coreboot 4.9, mainboard_vbt_filename() in src/mainboard/lenovo/x200/blc.c calls  get_blc_pwm_freq_value(NULL), making the following if condition always falling to the first, not to mention null dereference on strcmp() in get_blc_pwm_freq_value().

In order to determine the correct vbt filename at runtime, edid ascii string should be obtained first.

This issue may be distantly related to #474.



-- 
You have received this notification because you have either subscribed to it, or are involved in it.
To change your notification preferences, please click here: https://ticket.coreboot.org/my/account
_______________________________________________
coreboot mailing list -- [email protected]
To unsubscribe send an email to [email protected]
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.