Fix recursive NewArrayBufferWithContents (#529)

jschwe · web-flow · commit e299b24d2d48 · 2024-12-09T13:51:23.000Z
Signed-off-by: Jonathan Schwender &lt;schwenderjonathan@gmail.com&gt;
diff --git a/mozjs-sys/src/jsapi.cpp b/mozjs-sys/src/jsapi.cpp
@@ -158,7 +158,7 @@ JSObject* NewExternalArrayBuffer(JSContext* cx, size_t nbytes, void* contents,
 JSObject* NewArrayBufferWithContents(JSContext* cx, size_t nbytes,
                                      void* contents) {
   js::UniquePtr<void, JS::FreePolicy> dataPtr{contents};
-  return NewArrayBufferWithContents(cx, nbytes, contents);
+  return JS::NewArrayBufferWithContents(cx, nbytes, std::move(dataPtr));
 }
 
 // Reexport some methods

Original file line number	Diff line number	Diff line change
`@@ -158,7 +158,7 @@ JSObject* NewExternalArrayBuffer(JSContext* cx, size_t nbytes, void* contents,`
`158`	`158`	`JSObject* NewArrayBufferWithContents(JSContext* cx, size_t nbytes,`
`159`	`159`	`void* contents) {`
`160`	`160`	`js::UniquePtr<void, JS::FreePolicy> dataPtr{contents};`
`161`		`- return NewArrayBufferWithContents(cx, nbytes, contents);`
	`161`	`+ return JS::NewArrayBufferWithContents(cx, nbytes, std::move(dataPtr));`
`162`	`162`	`}`
`163`	`163`
`164`	`164`	`// Reexport some methods`