commit 9d723d3d573a53173bd8210be20f0ec622eef8fd
author Svet Ganov <svetoslavganov@google.com> Sat Aug 27 11:05:56 2016 -0700
committer Svet Ganov <svetoslavganov@google.com> Mon Aug 29 10:51:08 2016 -0700
tree d9dbb591fffb3ae5cc2998f1cfee80634686b41e
parent feef989771c8b4393e0c742097390bc95c1ba4fb

Polish MemoryIntArray

1. Add close guard
2. Adopt instead of clone the ahsmem fd to fix a dangling fd
3. Clear only the return flag when writing fd to parcel
4. Immediately destroy remote MemoryIntArray if stale
5. Throw Java exception if someone closed the fd under us

Change-Id: I85533fec336c40e3380e10d5448e18c9616ec341

core/java/android/provider/Settings.java

diff --git a/core/java/android/provider/Settings.java b/core/java/android/provider/Settings.java
index a7ff552..879f938 100755
--- a/core/java/android/provider/Settings.java
+++ b/core/java/android/provider/Settings.java
@@ -1658,6 +1658,9 @@
                                                     + cr.getPackageName() + " and user:"
                                                     + userHandle + " with index:" + index);
                                         }
+                                        if (mGenerationTracker != null) {
+                                            mGenerationTracker.destroy();
+                                        }
                                         mGenerationTracker = new GenerationTracker(array, index,
                                                 generation, () -> {
                                             synchronized (NameValueCache.this) {