This seems trivial, as internal_init() already does the right thing when a return value is used here.<div><br></div><div>Acked-by: David Hendricks <<a href="mailto:dhendrix@google.com">dhendrix@google.com</a>><br><br>

<div class="gmail_quote">On Wed, May 30, 2012 at 11:48 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">

Replace with a return instead, all other plumbing is already in place<br>
<br>
Signed-off-by: Niklas Söderlund <<a href="mailto:niso@kth.se">niso@kth.se</a>><br>
---<br>
 internal.c |    2 +-<br>
 1 file changed, 1 insertion(+), 1 deletion(-)<br>
<br>
diff --git a/internal.c b/internal.c<br>
index 5b24577..75e526b 100644<br>
--- a/internal.c<br>
+++ b/internal.c<br>
@@ -298,7 +298,7 @@ int internal_init(void)<br>
                        msg_perr("Proceeding anyway because user forced us to.\n");<br>
                } else {<br>
                        msg_perr("Aborting.\n");<br>
-                       exit(1);<br>
+                       return 1;<br>
                }<br>
        }<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
1.7.10.2<br>
<br>
<br>
_______________________________________________<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></font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br>David Hendricks (dhendrix)<br>

Systems Software Engineer, Google Inc.<br>
</div>