tls: do not leak WriteWrap objects

Kill WriteWrap instances that are allocated in `tls_wrap.cc` internally. Fix: #1075 PR-URL: #1090 Reviewed-By: Ben Noordhuis <[email protected]>
nodejs · Mar 7, 2015 · 7f4c95e · 7f4c95e
1 parent e763220
commit 7f4c95e
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/src/tls_wrap.cc b/src/tls_wrap.cc
@@ -305,6 +305,7 @@ void TLSWrap::EncOut() {
   for (size_t i = 0; i < count; i++)
     buf[i] = uv_buf_init(data[i], size[i]);
   int r = stream_->DoWrite(write_req, buf, count, nullptr);
+  write_req->Dispatched();
 
   // Ignore errors, this should be already handled in js
   if (!r)
@@ -314,6 +315,8 @@ void TLSWrap::EncOut() {
 
 void TLSWrap::EncOutCb(WriteWrap* req_wrap, int status) {
   TLSWrap* wrap = req_wrap->wrap()->Cast<TLSWrap>();
+  req_wrap->~WriteWrap();
+  delete[] reinterpret_cast<char*>(req_wrap);
 
   // Handle error
   if (status) {