On Sun, 23 Sep 2012 01:30:50 +0200
Carl-Daniel Hailfinger <[email protected]> wrote:

> > +static int dmi_shutdown(void *data)
> > +{
> > +   int i;
> > +   for (i = 0; i < ARRAY_SIZE(dmistrings); i++) {
> > +           free(dmistrings[i]);  
> 
> libflashrom allows calling programmer_init() and programmer_shutdown()
> again after one flashrom run. For that case, we want the second run to
> have zeroed dmistrings[i]. Please insert
> dmistrings[i] = NULL;

IIRC i even had this in first. it is not necessary though because they
get overwritten by dmi_init() which gets called unconditionally by
internal_init(). i am not convinced, that it is a good idea to null
them anyway. note that the array is not initialized at startup either.

-- 
Kind regards/Mit freundlichen Grüßen, Stefan Tauner

_______________________________________________
flashrom mailing list
[email protected]
http://www.flashrom.org/mailman/listinfo/flashrom

Reply via email to