<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
</head>
<body>
<div class="moz-cite-prefix">On 31/10/2020 15:15, Scott Q. wrote:<br>
</div>
<blockquote type="cite"
cite="mid:ee7b01050d37f421f67d428918136474@polarismail--com.w.emailarray.com">
<meta http-equiv="content-type" content="text/html; charset=UTF-8">
<style type="text/css">
body,p,td,div,span{
font-size:14px;font-family: -apple-system, BlinkMacSystemFont, "Segoe UI", Roboto, Helvetica, Arial, sans-serif, "Apple Color Emoji", "Segoe UI Emoji", "Segoe UI Symbol";
}</style>
But do you know how this bug was introduced ?
<div><br>
</div>
<div>I checked the history of that file and don't see anything
introduced recently that might cause this: <a
href="https://github.com/dovecot/core/commits/master/src/lib-http/http-client-request.c"
class="normal-link normal-link-url" target="_blank"
rel="noopener noreferrer" moz-do-not-send="true">https://github.com/dovecot/core/commits/master/src/lib-http/http-client-request.c</a></div>
<div><br>
</div>
<div>The only one that might be somewhat related is this one from
Apr 27 by Stephan Bosch</div>
<div><br>
</div>
<div><a
href="https://github.com/dovecot/core/commit/799b52accf71e86756dde738d22c1c6a500a7e29#diff-1c02b3481573ffd33c9abccd3f5a6752a5cd81ca83389f4380657f7309c06366"
class="normal-link normal-link-url" target="_blank"
rel="noopener noreferrer" moz-do-not-send="true">https://github.com/dovecot/core/commit/799b52accf71e86756dde738d22c1c6a500a7e29#diff-1c02b3481573ffd33c9abccd3f5a6752a5cd81ca83389f4380657f7309c06366</a></div>
<div><br>
</div>
</blockquote>
<p>I don't know exactly. I'm guess it's not the above commit, since
the problem is happening only when payload_finished is TRUE on
arriving at the start of the <span class="pl-en">http_client_request_send_more
function </span>and those lines set it to FALSE. Though I can't
exclude it that it might be an indirect effect of this.<br>
</p>
<p>There are two places where http_client_request_send_more is
called but the problematic call is the one that occurs in
http-client-connection.c within the
http_client_connection_continue_request function. <br>
</p>
<p>I wasn't able to identify a specific commit that introduced this
problem. The http-client-connection.c file and the whole library
has undergone numerous non trivial commits between 2.3.10 and
2.3.11.3 so the answer is not easy to identify by code
inspectiion. With the various changes, the code now gets to <span
class="pl-en">http_client_request_send_more with
payload_finished true but with payload_input NULL whereas in
2.3.10 payload_input was not NULL. (It's a logical deduction
from the fact we didn't see the assert failures before and that
part of the code has not changed).<br>
</span></p>
<p><span class="pl-en">John<br>
</span></p>
<p><span class="pl-en"></span></p>
<p><br>
</p>
</body>
</html>