Merge pull request #147 from verhovsky/getChangedRanges-mem-leak

verhovsky · web-flow · commit 3224eefcd775 · 2023-06-25T12:19:32.000+01:00
Fix memory leak in getChangedRanges()
diff --git a/src/tree.cc b/src/tree.cc
@@ -143,6 +143,8 @@ void Tree::GetChangedRanges(const Nan::FunctionCallbackInfo<Value> &info) {
     Nan::Set(result, i, RangeToJS(ranges[i]));
   }
 
+  free(ranges);
+
   info.GetReturnValue().Set(result);
 }
 

Original file line number	Diff line number	Diff line change
`@@ -143,6 +143,8 @@ void Tree::GetChangedRanges(const Nan::FunctionCallbackInfo<Value> &info) {`
`143`	`143`	`Nan::Set(result, i, RangeToJS(ranges[i]));`
`144`	`144`	`}`
`145`	`145`
	`146`	`+ free(ranges);`
	`147`	`+`
`146`	`148`	`info.GetReturnValue().Set(result);`
`147`	`149`	`}`
`148`	`150`