Drop the O_NOFOLLOW flag.
It's unclear why it was put in the first pinner implementation, and we
may now get symbolic links returned by ART's metadata files.
Test: boot and see that there's no pinner warning
Bug: 185833667
Change-Id: I6cb1a5a9f684c7f0e61bb7fca394030d5dd30c58
diff --git a/services/core/java/com/android/server/PinnerService.java b/services/core/java/com/android/server/PinnerService.java
index 1241b7779..53af88e 100644
--- a/services/core/java/com/android/server/PinnerService.java
+++ b/services/core/java/com/android/server/PinnerService.java
@@ -848,9 +848,7 @@
int mapSize = 0;
try {
- int openFlags = (OsConstants.O_RDONLY |
- OsConstants.O_CLOEXEC |
- OsConstants.O_NOFOLLOW);
+ int openFlags = (OsConstants.O_RDONLY | OsConstants.O_CLOEXEC);
fd = Os.open(fileToPin, openFlags, 0);
mapSize = (int) Math.min(Os.fstat(fd).st_size, Integer.MAX_VALUE);
address = Os.mmap(0, mapSize,