[M] Change in flashrom[master]: flashrom.c: Remove programmer_param global state

[Angel Pons (Code Review)]

Attention is currently required from: Edward O'Callaghan.

Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/66659 )

Change subject: flashrom.c: Remove programmer_param global state
......................................................................

Patch Set 15:

(2 comments)

File flashrom.c:

https://review.coreboot.org/c/flashrom/+/66659/comment/279da419_3fa3693b 
PS15, Line 171: 	const struct programmer_cfg cfg = { .params = strdup(param) };
Where did the NULL check for `param` go? Why is strdup() needed now? And what if `strdup()` returns NULL?

https://review.coreboot.org/c/flashrom/+/66659/comment/f17d12ce_eedaee6e 
PS15, Line 174: 	free(cfg.params);
Doesn't this result in use-after-free problems?