[prev in list] [next in list] [prev in thread] [next in thread] 

List:       linuxbios
Subject:    [coreboot] [coreboot - Bug #475] (New) mainboard_vbt_filename() in src/mainboard/lenovo/x200/blc.c d
From:       Bill XIE <coreboot () fe80 ! eu>
Date:       2023-03-30 12:15:49
Message-ID: redmine.issue-475.20230330121549.700 () fe80 ! eu
[Download RAW message or body]

Issue #475 has been reported by Bill XIE.

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

* Author: Bill XIE
* Status: New
* Priority: Normal
* Assignee: Arthur Heymans
* Category: board support
* Target version: master
* Start date: 2023-03-30
* Affected versions: 4.15, 4.16, 4.17, 4.18, 4.19, master
* 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 -- \
coreboot@coreboot.org To unsubscribe send an email to coreboot-leave@coreboot.org


[prev in list] [next in list] [prev in thread] [next in thread]