Fix getMany() memory leak (#193)

vweevers · web-flow · commit 680f94b40f60 · 2022-03-25T12:22:07.000+01:00
Ref #192 Ref Level/leveldown#790
diff --git a/binding.cc b/binding.cc
@@ -274,6 +274,7 @@ static std::vector<std::string>* KeyArray (napi_env env, napi_value arr) {
           StringOrBufferLength(env, element) > 0) {
         LD_STRING_OR_BUFFER_TO_COPY(env, element, to);
         result->emplace_back(toCh_, toSz_);
+        delete [] toCh_;
       }
     }
   }

Original file line number	Diff line number	Diff line change
`@@ -274,6 +274,7 @@ static std::vector<std::string>* KeyArray (napi_env env, napi_value arr) {`
`274`	`274`	`StringOrBufferLength(env, element) > 0) {`
`275`	`275`	`LD_STRING_OR_BUFFER_TO_COPY(env, element, to);`
`276`	`276`	`result->emplace_back(toCh_, toSz_);`
	`277`	`+ delete [] toCh_;`
`277`	`278`	`}`
`278`	`279`	`}`
`279`	`280`	`}`