https://redmine.lighttpd.net/https://redmine.lighttpd.net/favicon.ico?13667327412014-03-27T12:05:11Zlighty labsLighttpd - Bug #2564: strtol() usagehttps://redmine.lighttpd.net/issues/2564?journal_id=83482014-03-27T12:05:11Zdarix
<ul></ul><p>use <pre>server.port = 80</pre></p> Lighttpd - Bug #2564: strtol() usagehttps://redmine.lighttpd.net/issues/2564?journal_id=83492014-03-27T12:06:56ZOlaf-van-der-Spek
<ul></ul><p>darix wrote:</p>
<blockquote>
<p>use [...]</p>
</blockquote>
<p>I know, just pointing out an inconsistency.</p> Lighttpd - Bug #2564: strtol() usagehttps://redmine.lighttpd.net/issues/2564?journal_id=88752016-02-22T22:36:05Zgstrauss
<ul></ul><p>While changes such as this are low priority, consistency is good.</p>
<p>Submitted pull request <a class="external" href="https://github.com/lighttpd/lighttpd1.4/pull/26">https://github.com/lighttpd/lighttpd1.4/pull/26</a></p> Lighttpd - Bug #2564: strtol() usagehttps://redmine.lighttpd.net/issues/2564?journal_id=88892016-02-23T16:42:44Zstbuehler
<ul><li><strong>Target version</strong> set to <i>1.4.x</i></li></ul><p>The question is whether surrounding whitespace should actually be rejected - and given that we accepted it for some time I'd rather not change that.</p> Lighttpd - Bug #2564: strtol() usagehttps://redmine.lighttpd.net/issues/2564?journal_id=89012016-02-24T04:39:38Zgstrauss
<ul></ul><p>Would you accept the patch if I modify the patch hunk for configparser.y to issue a warning instead of an error? Checking strtol() for errors is not a bad thing. Whether or not to propagate the error is the question, and you have shared that you prefer not change existing behavior to propagate the error for config parsing. There are other uses of strtol() where some extra checks can improve robustness of the code.</p> Lighttpd - Bug #2564: strtol() usagehttps://redmine.lighttpd.net/issues/2564?journal_id=89252016-02-27T15:43:08Zgstrauss
<ul></ul><p>Updated <a class="external" href="https://github.com/lighttpd/lighttpd1.4/pull/26">https://github.com/lighttpd/lighttpd1.4/pull/26</a> to change config parsing strtol() error to a warning.</p> Lighttpd - Bug #2564: strtol() usagehttps://redmine.lighttpd.net/issues/2564?journal_id=91712016-03-26T11:00:05Zstbuehler
<ul><li><strong>Status</strong> changed from <i>New</i> to <i>Fixed</i></li><li><strong>% Done</strong> changed from <i>0</i> to <i>100</i></li></ul><p>Applied in changeset r3122.</p> Lighttpd - Bug #2564: strtol() usagehttps://redmine.lighttpd.net/issues/2564?journal_id=91722016-03-26T11:02:16Zstbuehler
<ul><li><strong>Target version</strong> changed from <i>1.4.x</i> to <i>1.4.40</i></li></ul>