Fix atom leak in JS_NewSymbol

engsr6982 · engsr6982 · commit 294f198aab64 · 2026-04-15T00:20:28.000+08:00
diff --git a/quickjs.c b/quickjs.c
@@ -3415,6 +3415,7 @@ JSValue JS_NewSymbol(JSContext *ctx, const char *description, bool is_global)
     JSAtom atom = JS_NewAtom(ctx, description);
     if (atom == JS_ATOM_NULL)
         return JS_EXCEPTION;
+    JS_FreeAtom(ctx, atom);
     return JS_NewSymbolFromAtom(ctx, atom, is_global ? JS_ATOM_TYPE_GLOBAL_SYMBOL : JS_ATOM_TYPE_SYMBOL);
 }
 

Original file line number	Diff line number	Diff line change
`@@ -3415,6 +3415,7 @@ JSValue JS_NewSymbol(JSContext ctx, const char description, bool is_global)`
`3415`	`3415`	`JSAtom atom = JS_NewAtom(ctx, description);`
`3416`	`3416`	`if (atom == JS_ATOM_NULL)`
`3417`	`3417`	`return JS_EXCEPTION;`
	`3418`	`+ JS_FreeAtom(ctx, atom);`
`3418`	`3419`	`return JS_NewSymbolFromAtom(ctx, atom, is_global ? JS_ATOM_TYPE_GLOBAL_SYMBOL : JS_ATOM_TYPE_SYMBOL);`
`3419`	`3420`	`}`
`3420`	`3421`