Fix leaked proxy_execute_jsc_get_proxies_for_url() if exception raised

[sergio-nsk]

No description provided.

[codecov]

Codecov Report

Attention: Patch coverage is 50.00000% with 6 lines in your changes missing coverage. Please review.

Project coverage is 59.28%. Comparing base (566dd35) to head (a88e5f8).
Report is 1 commits behind head on master.

Files with missing lines	Patch %	Lines
execute_jsc.c	50.00%	1 Missing and 5 partials ⚠️

Additional details and impacted files

@@           Coverage Diff           @@
##           master      #86   +/-   ##
=======================================
  Coverage   59.28%   59.28%           
=======================================
  Files          54       54           
  Lines        4883     4888    +5     
  Branches     1116     1118    +2     
=======================================
+ Hits         2895     2898    +3     
- Misses       1359     1360    +1     
- Partials      629      630    +1     

Flag	Coverage Δ
macos	55.87% <ø> (+0.11%)	⬆️
ubuntu	55.30% <50.00%> (+<0.01%)	⬆️
ubuntu_curl	57.34% <50.00%> (-0.04%)	⬇️
windows	56.84% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[sergio-nsk]

Removed usages of g_clean_object because this is a macro calling two functions g_clear_pointer() and g_object_unref(). I don't want to dlsym(module, "g_clear_pointer") and redefine the macro for calling the dynamically bound functions.

[nmoinvaz]

Do we also need to do this in JavaScriptCore one? Load it as a dependency of the module?

[sergio-nsk]

JavaScriptCore uses JSGlobalContextRelease().