Skip to content

Commit c7eff61

Browse files
authored
dns: fix parse memory leaky
PR-URL: #58973 Reviewed-By: Zeyu "Alex" Yang <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Marco Ippolito <[email protected]> Reviewed-By: James M Snell <[email protected]>
1 parent 41cd6ba commit c7eff61

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

src/cares_wrap.cc

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1565,6 +1565,8 @@ Maybe<int> SoaTraits::Parse(QuerySoaWrap* wrap,
15651565

15661566
if (status != ARES_SUCCESS) return Just<int>(status);
15671567

1568+
auto cleanup = OnScopeLeave([&]() { ares_free_data(soa_out); });
1569+
15681570
Local<Object> soa_record = Object::New(env->isolate());
15691571

15701572
if (soa_record
@@ -1605,8 +1607,6 @@ Maybe<int> SoaTraits::Parse(QuerySoaWrap* wrap,
16051607
return Nothing<int>();
16061608
}
16071609

1608-
ares_free_data(soa_out);
1609-
16101610
wrap->CallOnComplete(soa_record);
16111611
return Just<int>(ARES_SUCCESS);
16121612
}

0 commit comments

Comments
 (0)