Change in flashrom[master]: dummyflasher.c: Fix data leak in params processing error paths

[Anastasia Klimchuk (Code Review)]

Attention is currently required from: Angel Pons, Anastasia Klimchuk.
Hello build bot (Jenkins), Nico Huber, Namyoon Woo, Edward O'Callaghan, Angel Pons,

I'd like you to reexamine a change. Please visit

https://review.coreboot.org/c/flashrom/+/54748

to look at the new patch set (#5).

Change subject: dummyflasher.c: Fix data leak in params processing error paths
......................................................................

dummyflasher.c: Fix data leak in params processing error paths

This patch extracts params processing into a separate function. Now
all error paths of params processing return 1 back to init function
which frees data.

And there was just one more error path in init function were free(data)
needed to be added.

This is a follow up on CB:45230 which moves global state into
spi_master data.

A good side-effect of the change is: init function becomes easier
to read.

BUG=b:185191942
TEST=ninja test

Change-Id: I04f55f77bb4703f1d88b2191c45a22be3c97bf87
Signed-off-by: Anastasia Klimchuk 
---
M dummyflasher.c
1 file changed, 51 insertions(+), 27 deletions(-)

git pull ssh://review.coreboot.org:29418/flashrom refs/changes/48/54748/5