
<html>

  <head>

    <meta content="text/html; charset=utf-8" http-equiv="Content-Type">

  </head>

  <body bgcolor="#FFFFFF" text="#000000">

    <br>

    <div class="moz-cite-prefix">On 14.02.2017 04:22, Amos Jeffries

      wrote:<br>

    </div>

    <blockquote

      cite="mid:4c5ca6b2-dfc7-a1c3-be01-973373ae3b89@treenet.co.nz"

      type="cite">

      <pre wrap="">The problem is with proxy where the admin has configured large headers

to be allowed, and receives a Via just under the 6KB liit. Our append

pushing it over by even one byte would assert.</pre>

    </blockquote>

    <br>

    I am attaching a patch using SBuf instead of String for Via<br>

    accumulating, as you previously suggested. I am not<br>

    sure this is a much better solution since we have to do an<br>

    extra copy into SBuf.<br>

    <br>

    > The older bbuf code<br>

    > cropping at 1KB was nasty but would not crash Squid.<br>

    <br>

    I don't see the code preventing Squid from String overflow<br>

    there. The bbuf you are talking about was appended to via string<br>

    by strListAdd() without any checks...<br>

    <br>

    <br>

    Thanks,<br>

    Eduard.<br>

    <br class="Apple-interchange-newline">

  </body>

</html>