commit 9987b74e5a11b910c0de15f02491a101cc763c4f
author Vladimir Marko <vmarko@google.com> Thu Feb 06 11:47:52 2020 +0000
committer Vladimir Marko <vmarko@google.com> Thu Feb 06 15:48:46 2020 +0000
tree 51543be8c4840eed4c37f5925b8fe1a045dc8efe
parent f6afcc9bd7f9b82df5e93d96d89c4c8da898e6f0

Fix memfd_create_compat() F_DUPFD call.

Pass 0 as the lowest required fd arg for F_DUPFD, instead
of picking up uninitialized garbage.

Test: dex2oat_image_test on fugu.
Bug: 149002515
Change-Id: I2af21d47430133eba90877e7cf35424597fb78ce

libartbase/base/memfd.cc

diff --git a/libartbase/base/memfd.cc b/libartbase/base/memfd.cc
index 2aab7fc..8512a3a 100644
--- a/libartbase/base/memfd.cc
+++ b/libartbase/base/memfd.cc
@@ -89,7 +89,7 @@
       // We want the normal 'dup' semantics since memfd_create without any flags isn't CLOEXEC.
       // Unfortunately on some android targets we will compiler error if we use dup directly and so
       // need to use fcntl.
-      int nfd = fcntl(fileno(file), F_DUPFD);
+      int nfd = fcntl(fileno(file), F_DUPFD, /*lowest allowed fd*/ 0);
       fclose(file);
       return nfd;
     }