Skip to content

Commit 6cbddc3

Browse files
committed
synchronized with lua-nginx-module #23e69fb.
bugfix: ngx_http_lua_ffi_ssl_set_serialized_session(): avoided memory leak when calling it repeatly. Thanks spacewander for the upstream patch.
1 parent 157aaa9 commit 6cbddc3

File tree

1 file changed

+6
-0
lines changed

1 file changed

+6
-0
lines changed

src/http/ngx_http_lua_ssl_session_fetchby.c

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -569,6 +569,7 @@ ngx_http_lua_ffi_ssl_set_serialized_session(ngx_http_request_t *r,
569569
ngx_ssl_conn_t *ssl_conn;
570570
ngx_connection_t *c;
571571
ngx_ssl_session_t *session = NULL;
572+
ngx_ssl_session_t *old_session;
572573
ngx_http_lua_ssl_ctx_t *cctx;
573574

574575
c = r->connection;
@@ -599,8 +600,13 @@ ngx_http_lua_ffi_ssl_set_serialized_session(ngx_http_request_t *r,
599600
return NGX_ERROR;
600601
}
601602

603+
old_session = cctx->session;
602604
cctx->session = session;
603605

606+
if (old_session != NULL) {
607+
ngx_ssl_free_session(old_session);
608+
}
609+
604610
return NGX_OK;
605611
}
606612

0 commit comments

Comments
 (0)