<div dir="ltr">LGTM</div><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Feb 4, 2018 at 8:49 PM, 洪志道 <span dir="ltr"><<a href="mailto:hongzhidao@gmail.com" target="_blank">hongzhidao@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi!<div><br></div><div>Maybe it's better to keep the options more complete, and consider the follow operation as a mistake.</div><div><br></div><div>/usr/local/nginx/sbin/nginx -</div><div><div><br></div><div><div>hg diff</div><div>diff -r cbf59d483c9c src/core/nginx.c</div><div>--- a/src/core/nginx.c<span style="white-space:pre-wrap">   </span>Tue Jan 16 13:52:03 2018 +0300</div><div>+++ b/src/core/nginx.c<span style="white-space:pre-wrap">     </span>Sun Feb 04 07:41:36 2018 -0500</div><div>@@ -753,7 +753,12 @@</div><div>             return NGX_ERROR;</div><div>         }</div><div><br></div><div>-        while (*p) {</div><div>+        do {</div><div>+</div><div>+            if (*p == '\0') {</div><div>+                ngx_log_stderr(0, "missing option: \"%s\"", argv[i]);</div><div>+                return NGX_ERROR;</div><div>+            }</div><div><br></div><div>             switch (*p++) {</div><div><br></div><div>@@ -855,7 +860,8 @@</div><div>                 ngx_log_stderr(0, "invalid option: \"%c\"", *(p - 1));</div><div>                 return NGX_ERROR;</div><div>             }</div><div>-        }</div><div>+</div><div>+        } while (*p);</div><div><br></div><div>     next:</div></div></div><div><br></div><div>Thanks.</div></div>
<br>______________________________<wbr>_________________<br>
nginx-devel mailing list<br>
<a href="mailto:nginx-devel@nginx.org">nginx-devel@nginx.org</a><br>
<a href="http://mailman.nginx.org/mailman/listinfo/nginx-devel" rel="noreferrer" target="_blank">http://mailman.nginx.org/<wbr>mailman/listinfo/nginx-devel</a><br></blockquote></div><br></div>