js_uifc.c

7 years ago

Address Coverity defects 33325-33480, etc. Resource Leak: · c1754e19

rswindell authored 7 years ago

HANDLE_PENDING() contains a return, so it's theoretically possible that the
memory allocated by the previous JSVALUE/STRING_TO_... allocation could
be leaked. So now we pass an optional pointer to HANDLE_PENDING() which will
call free() on it if it's not NULL, and then sets it to NULL for good measure.

c1754e19

History

Address Coverity defects 33325-33480, etc. Resource Leak:

rswindell authored 7 years ago

HANDLE_PENDING() contains a return, so it's theoretically possible that the
memory allocated by the previous JSVALUE/STRING_TO_... allocation could
be leaked. So now we pass an optional pointer to HANDLE_PENDING() which will
call free() on it if it's not NULL, and then sets it to NULL for good measure.