Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit b8b8e215 authored by Christopher Ferris's avatar Christopher Ferris Committed by Colin Cross
Browse files

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
Merged-In: Ie88e2fac76ba0da92c434d2db107885253855696
(cherry picked from commit 60e4b9c4)
parent 0d801d9e

core/jni/fd_utils.cpp

+1 −0
Original line number Diff line number Diff line
@@ -580,6 +580,7 @@ void FileDescriptorTable::RestatInternal(std::set<int>& 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