Seems trivial, and I think you're also helping to make sure memory alloc'd in doit() gets free'd properly. So:<div><br></div><div>Acked-by: David Hendricks <<a href="mailto:dhendrix@google.com">dhendrix@google.com</a>><br>

<br>Thanks for doing this!</div><div><br><div class="gmail_quote">On Wed, May 30, 2012 at 11:21 AM, Niklas Söderlund <span dir="ltr"><<a href="mailto:niso@kth.se" target="_blank">niso@kth.se</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

Sorry I forgot,<br>
<br>
Signed-off-by: Niklas Söderlund <<a href="mailto:niso@kth.se">niso@kth.se</a>><br>
<div class="HOEnZb"><div class="h5"><br>
On 05/30/2012 08:18 PM, Niklas Söderlund wrote:<br>
<br>
> The only caller is able to check the return code and handle it correctly<br>
> ---<br>
>  flashrom.c |    5 ++++-<br>
>  layout.c   |    2 +-<br>
>  2 files changed, 5 insertions(+), 2 deletions(-)<br>
><br>
> diff --git a/flashrom.c b/flashrom.c<br>
> index ec8137b..f6597e5 100644<br>
> --- a/flashrom.c<br>
> +++ b/flashrom.c<br>
> @@ -1781,7 +1781,10 @@ int doit(struct flashctx *flash, int force, const char *filename, int read_it,<br>
><br>
>  #if CONFIG_INTERNAL == 1<br>
>               if (programmer == PROGRAMMER_INTERNAL)<br>
> -                     show_id(newcontents, size, force);<br>
> +                     if (show_id(newcontents, size, force)) {<br>
> +                             ret = 1;<br>
> +                             goto out;<br>
> +                     }<br>
>  #endif<br>
>       }<br>
><br>
> diff --git a/layout.c b/layout.c<br>
> index 379ee8c..0512638 100644<br>
> --- a/layout.c<br>
> +++ b/layout.c<br>
> @@ -137,7 +137,7 @@ int show_id(uint8_t *bios, int size, int force)<br>
>                                 "\n\n",<br>
>                                 mainboard_vendor, mainboard_part, lb_vendor,<br>
>                                 lb_part);<br>
> -                     exit(1);<br>
> +                     return 1;<br>
>               }<br>
>       }<br>
><br>
<br>
<br>
<br>
</div></div><div class="HOEnZb"><div class="h5">_______________________________________________<br>
flashrom mailing list<br>
<a href="mailto:flashrom@flashrom.org">flashrom@flashrom.org</a><br>
<a href="http://www.flashrom.org/mailman/listinfo/flashrom" target="_blank">http://www.flashrom.org/mailman/listinfo/flashrom</a><br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br>David Hendricks (dhendrix)<br>Systems Software Engineer, Google Inc.<br>
</div>