Hello everyone,<br><br>In ngx_http_gzip_disable function of http/ngx_http_core_module.c, <br>The code is here: <a href="http://www.google.com/codesearch/p?hl=en#N6mCnE4lf5g/src/http/ngx_http_core_module.c&q=http_gzip_disable%20package:nginx&sa=N&cd=1&ct=rc&l=73">http://www.google.com/codesearch/p?hl=en#N6mCnE4lf5g/src/http/ngx_http_core_module.c&q=http_gzip_disable%20package:nginx&sa=N&cd=1&ct=rc&l=73</a> Line 4387-4392: <br>
<pre><span id="l4387"> <span class="stx-keyword">for</span> (i = <span class="stx-number">1</span>; i < cf->args->nelts; i++) {
</span><span id="l4388">
</span><span id="l4389"> <span class="stx-keyword">if</span> (ngx_strcmp(value[<span class="stx-number">1</span>].data, <span class="stx-string">"msie6"</span>) == <span class="stx-number">0</span>) {
</span><span id="l4390"> clcf->gzip_disable_msie6 = <span class="stx-number">1</span>;
</span><span id="l4391"> <span class="stx-keyword">continue</span>;
</span><span id="l4392"> }
</span></pre>It used value[1] to compare with "msie6", which I think "1" is a misspell of "i". If it is indeed 1, then I think maybe the if statement should be located outside the for loop.<br>
<br>Thank you for taking the time to read.<br><br>Best regards.<br><br>hank<br>