diff options
author | Stefan Tauner <stefan.tauner@alumni.tuwien.ac.at> | 2014-07-13 12:52:15 +0000 |
---|---|---|
committer | Stefan Tauner <stefan.tauner@alumni.tuwien.ac.at> | 2014-07-13 12:52:15 +0000 |
commit | 9972d15e12a1706d280e427becd80d1f79675859 (patch) | |
tree | a2e8e2fd20756392fe86c8154dc610672c8e4f2d /README | |
parent | 440057a82e42c74c8769cfdd548fd93699bc3178 (diff) | |
download | flashrom-9972d15e12a1706d280e427becd80d1f79675859.tar.gz flashrom-9972d15e12a1706d280e427becd80d1f79675859.tar.bz2 flashrom-9972d15e12a1706d280e427becd80d1f79675859.zip |
Fix garbage handling in DMI strings
Previously we tried to replace garbage characters with <space> directly in
the read-only memory-mapped SMBIOS area(!). This could never have
worked for any DMI strings with garbage and results in a segfault on
machines with such strings.
Thanks to Brian Rak (Supermicro X10SLE-F) and John Pohlman (HP XW9400)
for reporting this issue.
With this patch the strings are duplicated within dmi_string() already,
just before we sanitize them. Also, the limit variable used everywhere
points to the first invalid byte address. Refine respective checks
accordingly.
Corresponding to flashrom svn r1824.
Signed-off-by: Stefan Tauner <stefan.tauner@alumni.tuwien.ac.at>
Acked-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006@gmx.net>
Diffstat (limited to 'README')
0 files changed, 0 insertions, 0 deletions