diff options
| author | Elliott Hughes <enh@google.com> | 2019-11-05 12:42:14 -0800 |
|---|---|---|
| committer | doc HD <doc.divxm@gmail.com> | 2020-04-23 14:54:03 +0300 |
| commit | 33249192591fb6dbe0bd301b44f09ac4de236713 (patch) | |
| tree | f1400fbc5b187fb17524b75168e68e130f765cfc | |
| parent | 7789e527153b9bf25ff3ad8ddd47fdb7632b1ed4 (diff) | |
android_getaddrinfo_proxy: fix memory leak on failure.q10.0
The shadowing of `ai` meant that the freeaddrinfo() call outside the
loop would never see anything but NULL.
Bug: https://issuetracker.google.com/143928781
Test: treehugger
Change-Id: I1bf137f7933201eb8024603bfd569ff7bbc7f9b7
| -rw-r--r-- | libc/dns/net/getaddrinfo.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/libc/dns/net/getaddrinfo.c b/libc/dns/net/getaddrinfo.c index 201b4f4b2..cc94b21e2 100644 --- a/libc/dns/net/getaddrinfo.c +++ b/libc/dns/net/getaddrinfo.c @@ -472,7 +472,7 @@ android_getaddrinfo_proxy( break; } - struct addrinfo* ai = calloc(1, sizeof(struct addrinfo) + sizeof(struct sockaddr_storage)); + ai = calloc(1, sizeof(struct addrinfo) + sizeof(struct sockaddr_storage)); if (ai == NULL) { break; } |