diff options
author | Anastasia Klimchuk <aklm@chromium.org> | 2021-04-21 15:19:24 +1000 |
---|---|---|
committer | Nico Huber <nico.h@gmx.de> | 2021-04-27 10:27:19 +0000 |
commit | 914aa7e60eff94864acc892d6228023e261524a8 (patch) | |
tree | 5e8df595c80ff56ff3c7dd06ee2d8a42076ded83 /libflashrom.c | |
parent | 62fb7649c89463ca045a7c4a376a096f7d4c2d63 (diff) | |
download | flashrom-914aa7e60eff94864acc892d6228023e261524a8.tar.gz flashrom-914aa7e60eff94864acc892d6228023e261524a8.tar.bz2 flashrom-914aa7e60eff94864acc892d6228023e261524a8.zip |
mec1308.c: Extract params check into a function
This allows char *p to become a local variable in check_params,
and it is allocated and freed within check_params function.
Which means init function does not need char *p anymore,
in particular does not need to free it - and this makes cleanup
after failed init easier.
As a good side effect, init function becomes easier to read.
TEST=builds and ninja test from 51487
BUG=b:185191942
Change-Id: If5be7709e93233a2e7ea9133de50382d2524a55f
Signed-off-by: Anastasia Klimchuk <aklm@chromium.org>
Reviewed-on: https://review.coreboot.org/c/flashrom/+/52595
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
Reviewed-by: Nico Huber <nico.h@gmx.de>
Diffstat (limited to 'libflashrom.c')
0 files changed, 0 insertions, 0 deletions