QuickJS: fix fs callback return value leak

Previously, the value returned by a callback passed to the callback
flavor of fs functions was never freed.  The value stayed referenced
until the runtime was freed, triggering the
"list_empty(&rt->gc_obj_list)" assertion in JS_FreeRuntime() in debug
builds and accumulating in the runtime with js_context_reuse.

The issue was introduced in 1f8f9992 (0.8.9).

diff --git a/external/qjs_fs_module.c b/external/qjs_fs_module.c
index 059b46eb39cc213b9508711768bb697a685a0cb9..42db4ff0b8569a5c200e6039725651a79edf0d22 100644 (file)
--- a/external/qjs_fs_module.c
+++ b/external/qjs_fs_module.c
@@ -2737,6 +2737,8 @@ qjs_fs_result(JSContext *cx, JSValue result, int calltype, JSValue callback)
             return JS_EXCEPTION;
         }
 
+        JS_FreeValue(cx, promise);
+
         return JS_UNDEFINED;
 
     default: