diff options
| author | Edward O'Callaghan <quasisec@google.com> | 2021-05-25 20:51:16 +1000 |
|---|---|---|
| committer | Edward O'Callaghan <quasisec@chromium.org> | 2021-05-27 02:35:43 +0000 |
| commit | 0282a83ac6953abdeadb9be034f67abf54f6decd (patch) | |
| tree | 7131f615fa1d991dd99c3c29f76ebcb418c7efd0 /atavia.c | |
| parent | 52c0275c706e71b11c497e86a1974855043362c3 (diff) | |
| download | flashrom-0282a83ac6953abdeadb9be034f67abf54f6decd.tar.gz flashrom-0282a83ac6953abdeadb9be034f67abf54f6decd.tar.bz2 flashrom-0282a83ac6953abdeadb9be034f67abf54f6decd.zip | |
dummyflasher.c: Get rid of get_data_from_context()
Relying on the global state 'dummy_buses_supported' to
determine the member master struct [mst.par or mst.spi]
is both buggy and ultimately unnecessary. It became
apparent after commit 4eef651ff503f81b77 just how fragile
this really was as the 'defaults' simultaneously selected
both buses causing get_data_from_context() to fall-though
however memory happened to workout by chance due to the
union. With the member master structs now being struct
fields the subtle bug is more apparent.
BUG=none
BRANCH=none
TEST=`./flashrom -r /tmp/fwupdater.apnSQQ -p dummy:emulate=VARIABLE_SIZE,image=test_update.sh.tmp.emu,size=8388608`
Change-Id: I07a34faf50ff0679cb3d6bc683142f82160010b1
Signed-off-by: Edward O'Callaghan <quasisec@google.com>
Reviewed-on: https://review.coreboot.org/c/flashrom/+/54907
Reviewed-by: Anastasia Klimchuk <aklm@chromium.org>
Reviewed-by: Sam McNally <sammc@google.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'atavia.c')
0 files changed, 0 insertions, 0 deletions