From 60e4b9c43c8985a96ececf8b15e1f7e44f6b4c8f Mon Sep 17 00:00:00 2001 From: Christopher Ferris Date: Tue, 3 May 2022 16:05:16 -0700 Subject: Fix memory leak of FileDescriptorInfo object. Bug: 231366539 Test: After fixing, there is no 192 byte leak present in most zygote Test: forked processes. Change-Id: Ie88e2fac76ba0da92c434d2db107885253855696 --- core/jni/fd_utils.cpp | 1 + 1 file changed, 1 insertion(+) diff --git a/core/jni/fd_utils.cpp b/core/jni/fd_utils.cpp index 40f6e4f63cd7..5c71f692b80f 100644 --- a/core/jni/fd_utils.cpp +++ b/core/jni/fd_utils.cpp @@ -580,6 +580,7 @@ void FileDescriptorTable::RestatInternal(std::set& open_fds, fail_fn_t fail // TODO(narayan): This will be an error in a future android release. // error = true; // ALOGW("Zygote closed file descriptor %d.", it->first); + delete it->second; it = open_fd_map_.erase(it); } else { // The entry from the file descriptor table is still open. Restat -- cgit v1.2.3-59-g8ed1b