Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Commit b2b40bc

Browse files
Fix leak freeing array from javscript
1 parent 06b04c7 commit b2b40bc

File tree

1 file changed

+8
-5
lines changed

1 file changed

+8
-5
lines changed

generate/templates/partials/async_function.cc

Lines changed: 8 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -179,18 +179,21 @@ void {{ cppClassName }}::{{ cppFunctionName }}Worker::Execute() {
179179

180180
{%endif%}
181181

182+
cppClassName == 'Array'
183+
182184
{%each args|argsInfo as arg %}
183185
{%if not arg.isSelf %}
184186
{%if not arg.payloadFor %}
185-
{%if arg.cppClassName == 'GitStrarray' %}
186-
{%if not arg.isReturn %}
187+
{%if not arg.isReturn %}
188+
{%if arg.cppClassName == 'GitStrarray' %}
187189
{{ arg.freeFunctionName }}((git_strarray*)baton->{{ arg.name }});
188190
free((void*)baton->{{ arg.name }});
189191
{%endif%}
190-
{%elsif not arg.isCallbackFunction %}
191-
{%if not arg.isStructType %}
192+
{%elsif arg.cppClassName == 'Array' %}
193+
free((void*)baton->{{ arg.name }});
194+
{%elsif not arg.isCallbackFunction %}
192195
{%if not arg.isStructType %}
193-
{%if not arg.isReturn %}
196+
{%if not arg.isStructType %}
194197
{%if not arg.isPayload %}
195198
{%if arg.name %}
196199
{%if arg.cppClassName == 'String'%}

0 commit comments

Comments
 (0)