[flashrom-gerrit] Change in flashrom[master]: dummyflasher.c: Upstream ChromiumOS 'freq' param feature

[Edward O'Callaghan (Code Review)]

Edward O'Callaghan has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/45461 )

Change subject: dummyflasher.c: Upstream ChromiumOS 'freq' param feature
......................................................................

Patch Set 1:

(2 comments)

https://review.coreboot.org/c/flashrom/+/45461/1/dummyflasher.c 
File dummyflasher.c:

https://review.coreboot.org/c/flashrom/+/45461/1/dummyflasher.c@297 
PS1, Line 297: tmp
> I don't see this use of tmp being freed anywhere.

Agreed, typical cros fork seems buggy here so fixed those leaks now.

https://review.coreboot.org/c/flashrom/+/45461/1/dummyflasher.c@308 
PS1, Line 308: goto dummy_init_out;
> Should this be return 1 like the other error case?

yes.