Anastasia Klimchuk has uploaded this change for review. ( https://review.coreboot.org/c/flashrom/+/52719 )
Change subject: mec1308.c: Ensure programmer param variable is always initialised ......................................................................
mec1308.c: Ensure programmer param variable is always initialised
Programmer param variable is now declared inline with the code and is const, so that it cannot be used uninitialised.
BUG=b:185191942 TEST=builds and ninja test from 51487
Change-Id: I2d2b3039da2ef185cb31509b3901d56b428688b7 Signed-off-by: Anastasia Klimchuk aklm@chromium.org --- M mec1308.c 1 file changed, 1 insertion(+), 3 deletions(-)
git pull ssh://review.coreboot.org:29418/flashrom refs/changes/19/52719/1
diff --git a/mec1308.c b/mec1308.c index 0ec9be9..1530d9b 100644 --- a/mec1308.c +++ b/mec1308.c @@ -409,9 +409,7 @@ static int check_params(void) { int ret = 0; - char *p = NULL; - - p = extract_programmer_param("type"); + char *const p = extract_programmer_param("type"); if (p && strcmp(p, "ec")) { msg_pdbg("mec1308 only supports "ec" type devices\n"); ret = 1;