<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
</head>
<body>
<div class="moz-cite-prefix">On 31/10/2020 00:18, Scott Q. wrote:<br>
</div>
<blockquote type="cite"
cite="mid:eef949e0948322198a97ef66369e88f3@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>
I have implemented this change and the core dumps went away.
<div><br>
</div>
<div>It appears so far that it's indeed the right fix.</div>
<div><br>
</div>
<div>Has this change been introduced in 2.3.11.3 ? </div>
<div><br>
</div>
<div>Thanks!</div>
</blockquote>
<p>I've also been running it for 4 days without any further panics /
segfaults and no noticed adverse effects. I'd recommend the
dovecot team to consider including the patch in the next update.</p>
<p>John</p>
<p>diff -ur dovecot-2.3.11.3-orig/src/lib-http/http-client-request.c
dovecot-2.3.11.3/src/lib-http/http-client-request.c<br>
--- dovecot-2.3.11.3-orig/src/lib-http/http-client-request.c
2020-08-12 14:20:41.000000000 +0200<br>
+++ dovecot-2.3.11.3/src/lib-http/http-client-request.c 2020-10-27
13:06:09.352973130 +0100<br>
@@ -1229,12 +1229,12 @@<br>
const char *error;<br>
uoff_t offset;<br>
<br>
- i_assert(req->payload_input != NULL);<br>
- i_assert(req->payload_output != NULL);<br>
-<br>
if (req->payload_finished)<br>
return
http_client_request_finish_payload_out(req);<br>
<br>
+ i_assert(req->payload_input != NULL);<br>
+ i_assert(req->payload_output != NULL);<br>
+<br>
io_remove(&conn->io_req_payload);<br>
<br>
/* chunked ostream needs to write to the parent stream's
buffer */</p>
</body>
</html>