Fix getMany() memory leak (#9)

vweevers · web-flow · commit 00364c716459 · 2022-03-25T11:46:49.000+01:00
Ref Level/rocksdb#192 Ref Level/leveldown#790
diff --git a/binding.cc b/binding.cc
@@ -285,6 +285,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
`@@ -285,6 +285,7 @@ static std::vector<std::string>* KeyArray (napi_env env, napi_value arr) {`
`285`	`285`	`StringOrBufferLength(env, element) >= 0) {`
`286`	`286`	`LD_STRING_OR_BUFFER_TO_COPY(env, element, to);`
`287`	`287`	`result->emplace_back(toCh_, toSz_);`
	`288`	`+ delete [] toCh_;`
`288`	`289`	`}`
`289`	`290`	`}`
`290`	`291`	`}`