On 24.02.2010 14:28, Michael Karcher wrote: > Signed-off-by: Michael Karcher <[email protected]> > > diff --git a/dmi.c b/dmi.c > index c80a568..100767f 100644 > --- a/dmi.c > +++ b/dmi.c > @@ -54,55 +54,56 @@ char *dmistrings[DMI_ID_INVALID]; > [...] > + printf_debug("DMI string %s: \"%s\"\n", string_name, answerbuf); > + > + return strdup(answerbuf); > +} >
If strdup runs out of memory, we'll return NULL without warning that we ran out of memory. Same coding style nitpicks as with the other patch. I trust you to explain/fix this. Acked-by: Carl-Daniel Hailfinger <[email protected]> Regards, Carl-Daniel -- "I do consider assignment statements and pointer variables to be among computer science's most valuable treasures." -- Donald E. Knuth _______________________________________________ flashrom mailing list [email protected] http://www.flashrom.org/mailman/listinfo/flashrom
