walkero
/
nghttp2
1
0
Fork 0
Code Issues Pull Requests Projects Releases 1 Wiki Activity

nghttpx: Fix memory leak

This commit is contained in:
Tatsuhiro Tsujikawa 2017-10-29 21:46:12 +09:00
parent c4f8afcfde
commit f2b8edd1e2
1 changed files with 2 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/shrpx_mruby_module_env.cc
View File

@ -161,6 +161,7 @@ mrb_value env_get_tls_client_fingerprint(mrb_state *mrb, mrb_value self) {
// Fingerprint is SHA-256, so we need 32 bytes buffer. // Fingerprint is SHA-256, so we need 32 bytes buffer.
std::array<uint8_t, 32> buf; std::array<uint8_t, 32> buf;
auto slen = tls::get_x509_fingerprint(buf.data(), buf.size(), x); auto slen = tls::get_x509_fingerprint(buf.data(), buf.size(), x);
X509_free(x);
if (slen == -1) { if (slen == -1) {
mrb_raise(mrb, E_RUNTIME_ERROR, "could not compute client fingerprint"); mrb_raise(mrb, E_RUNTIME_ERROR, "could not compute client fingerprint");
} }
@ -192,6 +193,7 @@ mrb_value env_get_tls_client_subject_name(mrb_state *mrb, mrb_value self) {
auto &balloc = downstream->get_block_allocator(); auto &balloc = downstream->get_block_allocator();
auto name = tls::get_x509_subject_name(balloc, x); auto name = tls::get_x509_subject_name(balloc, x);
X509_free(x);
return mrb_str_new(mrb, name.c_str(), name.size()); return mrb_str_new(mrb, name.c_str(), name.size());
} }
} // namespace } // namespace