BUG/MEDIUM: httpclient: check if the httpclient was released in the IO handler
Upon a applet_release(), the applet can be scheduled again and a call to
the IO handler is still possible. When the struct httpclient is already
free the IO handler could try to access it.
This patch fixes the issue by setting svcctx to NULL in the
applet_release, and checking its value in the IO handler.
Must be backported as far as 2.5.
diff --git a/src/http_client.c b/src/http_client.c
index 2f9bac5..92a4ed6 100644
--- a/src/http_client.c
+++ b/src/http_client.c
@@ -707,6 +707,11 @@
uint32_t sz;
int ret;
+ /* The IO handler could be called after the release, so we need to
+ * check if hc is still there to run the IO handler */
+ if (!hc)
+ return;
+
while (1) {
/* required to stop */
@@ -1115,6 +1120,10 @@
httpclient_destroy(hc);
}
+ /* be sure not to use this ptr anymore if the IO handler is called a
+ * last time */
+ appctx->svcctx = NULL;
+
return;
}