[GitHub] trafficserver pull request #1024: TS-4858: fix memory leak of global_default...

Github user shinrich closed the pull request at: https://github.com/apache/trafficserver/pull/1024 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the

[GitHub] trafficserver pull request #1024: TS-4858: fix memory leak of global_default...

Github user jpeach commented on a diff in the pull request: https://github.com/apache/trafficserver/pull/1024#discussion_r82625751 --- Diff: iocore/net/SSLCertLookup.cc --- @@ -160,7 +160,40 @@ ticket_block_alloc(unsigned count) return ptr; }

[GitHub] trafficserver pull request #1024: TS-4858: fix memory leak of global_default...

Github user shinrich commented on a diff in the pull request: https://github.com/apache/trafficserver/pull/1024#discussion_r82625741 --- Diff: iocore/net/SSLUtils.cc --- @@ -2159,7 +2123,7 @@ ssl_callback_session_ticket(SSL *ssl, unsigned char *keyname, unsigned char *iv,

[GitHub] trafficserver pull request #1024: TS-4858: fix memory leak of global_default...

Github user jpeach commented on a diff in the pull request: https://github.com/apache/trafficserver/pull/1024#discussion_r82625392 --- Diff: iocore/net/SSLCertLookup.cc --- @@ -160,7 +160,40 @@ ticket_block_alloc(unsigned count) return ptr; }

[GitHub] trafficserver pull request #1024: TS-4858: fix memory leak of global_default...

Github user jpeach commented on a diff in the pull request: https://github.com/apache/trafficserver/pull/1024#discussion_r82625369 --- Diff: iocore/net/SSLConfig.cc --- @@ -243,6 +245,21 @@ SSLConfigParams::initialize() ats_free(ssl_server_ca_cert_filename);

[GitHub] trafficserver pull request #1024: TS-4858: fix memory leak of global_default...

Github user persiaAziz commented on a diff in the pull request: https://github.com/apache/trafficserver/pull/1024#discussion_r82054996 --- Diff: iocore/net/SSLCertLookup.cc --- @@ -160,7 +160,40 @@ ticket_block_alloc(unsigned count) return ptr; }

[GitHub] trafficserver pull request #1024: TS-4858: fix memory leak of global_default...

Github user jpeach commented on a diff in the pull request: https://github.com/apache/trafficserver/pull/1024#discussion_r82046421 --- Diff: iocore/net/SSLConfig.cc --- @@ -243,6 +245,21 @@ SSLConfigParams::initialize() ats_free(ssl_server_ca_cert_filename);

[GitHub] trafficserver pull request #1024: TS-4858: fix memory leak of global_default...

Github user jpeach commented on a diff in the pull request: https://github.com/apache/trafficserver/pull/1024#discussion_r82047142 --- Diff: iocore/net/SSLCertLookup.cc --- @@ -160,7 +160,40 @@ ticket_block_alloc(unsigned count) return ptr; }

[GitHub] trafficserver pull request #1024: TS-4858: fix memory leak of global_default...

Github user jpeach commented on a diff in the pull request: https://github.com/apache/trafficserver/pull/1024#discussion_r82049092 --- Diff: iocore/net/SSLUtils.cc --- @@ -2159,7 +2123,7 @@ ssl_callback_session_ticket(SSL *ssl, unsigned char *keyname, unsigned char *iv,

[GitHub] trafficserver pull request #1024: TS-4858: fix memory leak of global_default...

Github user jpeach commented on a diff in the pull request: https://github.com/apache/trafficserver/pull/1024#discussion_r82044347 --- Diff: iocore/net/P_SSLCertLookup.h --- @@ -110,5 +109,5 @@ struct SSLCertLookup : public ConfigInfo { void ticket_block_free(void *ptr);

[GitHub] trafficserver pull request #1024: TS-4858: fix memory leak of global_default...

Github user jpeach commented on a diff in the pull request: https://github.com/apache/trafficserver/pull/1024#discussion_r82046288 --- Diff: iocore/net/SSLConfig.cc --- @@ -243,6 +245,21 @@ SSLConfigParams::initialize() ats_free(ssl_server_ca_cert_filename);

[GitHub] trafficserver pull request #1024: TS-4858: fix memory leak of global_default...

Github user jpeach commented on a diff in the pull request: https://github.com/apache/trafficserver/pull/1024#discussion_r79039044 --- Diff: iocore/net/SSLUtils.cc --- @@ -2052,6 +2052,7 @@ SSLParseCertificateConfiguration(const SSLConfigParams *params, SSLCertLookup *l

[GitHub] trafficserver pull request #1024: TS-4858: fix memory leak of global_default...

GitHub user persiaAziz opened a pull request: https://github.com/apache/trafficserver/pull/1024 TS-4858: fix memory leak of global_default_keyblock fix memory leak of global_default_keyblock. global_defualt_keyblock will be freed and reassigned on each reload You can merge