diff options
author | Yann Ylavic <ylavic@apache.org> | 2021-10-12 16:48:18 +0000 |
---|---|---|
committer | Yann Ylavic <ylavic@apache.org> | 2021-10-12 16:48:18 +0000 |
commit | 59b7c104ce06c90be20ff50435d912a444341245 (patch) | |
tree | ba4b51a8953af2aa6c0035bfc50c65a7b0ec3d1f /changes-entries | |
parent | 17471dfb938c7e20681783939a147d433b468864 (diff) | |
download | httpd-59b7c104ce06c90be20ff50435d912a444341245.tar.gz |
*) core: Be safe with ap_lingering_close() called with a socket NULL-ed.
PR 65627.
mod_itk seems to:
ap_set_core_module_config(c->conn_config, NULL)
before calling ap_lingering_close(), causing a crash after r1891721.
Until we have an API to no-op ap_lingering_close(), let's be safe.
* server/connection.c(ap_start_lingering_close):
The socket should not be NULL here, add an assertion.
* server/connection.c(ap_lingering_close):
Set c->aborted if the socket is NULL, and give up.
Submitted by: acmondor <bz.apache.org acmondor.ca>, ylavic
git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1894171 13f79535-47bb-0310-9956-ffa450edef68
Diffstat (limited to 'changes-entries')
-rw-r--r-- | changes-entries/ap_lingering_close-NULL.txt | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/changes-entries/ap_lingering_close-NULL.txt b/changes-entries/ap_lingering_close-NULL.txt new file mode 100644 index 0000000000..43cc6930b5 --- /dev/null +++ b/changes-entries/ap_lingering_close-NULL.txt @@ -0,0 +1,3 @@ + *) core: Be safe with ap_lingering_close() called with a socket NULL-ed by + a third-party module. PR 65627. + [acmondor <bz.apache.org acmondor.ca>, Yann Ylavic] |