diff options
author | Andrey Suvorov <a.suvorov@f5.com> | 2021-05-17 14:28:38 -0700 |
---|---|---|
committer | Andrey Suvorov <a.suvorov@f5.com> | 2021-05-17 14:28:38 -0700 |
commit | 19dfeba86b9dda6f1960ba9b3dba4708565d27ad (patch) | |
tree | 416fa0e00149f6eb25b073d51f9ef868671c2d9b /src/nxt_freebsd_sendfile.c | |
parent | 1198118b3b987930c508d78d90af909eec1835db (diff) | |
download | unit-19dfeba86b9dda6f1960ba9b3dba4708565d27ad.tar.gz unit-19dfeba86b9dda6f1960ba9b3dba4708565d27ad.tar.bz2 |
Fixing a crash after applying the wrong TLS configuration.
When an invalid TLS configuration is applied (such as the conf_commands
feature), nxt_cert_store_get() creates a buffer to send a certificate request
to the main process and adds its default completion handler to an asynchronous
queue to free the allocated buffer. However, if configuration fails,
nxt_router_conf_error() removes the memory pool used to allocate the buffer,
causing a crash when the completion handler is dispatched.
Assertion "src/nxt_buf.c:208 assertion failed: data == b->parent" is triggered
when is NXT_DEBUG enabled in the configure script.
This patch uses a reference counter to retain the memory pool and redefines the
completion handler to free the buffer before releasing the memory pool.
Diffstat (limited to 'src/nxt_freebsd_sendfile.c')
0 files changed, 0 insertions, 0 deletions