From eaa0718780a29df8fbe42c59beecee5557fdf785 Mon Sep 17 00:00:00 2001
From: Joe Onorato <joeo@android.com>
Date: Thu, 23 Sep 2010 16:49:20 -0700
Subject: [PATCH] Crash apps when they provide bad notifications instead of
 crashing the system process.

Bug: 2958415
Change-Id: I9cbef340fb20a64eca8493950b814399c26ce197
---
 services/java/com/android/server/am/ServiceRecord.java | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/services/java/com/android/server/am/ServiceRecord.java b/services/java/com/android/server/am/ServiceRecord.java
index f35a68e734d4c..e5aceb45c4d9c 100644
--- a/services/java/com/android/server/am/ServiceRecord.java
+++ b/services/java/com/android/server/am/ServiceRecord.java
@@ -347,7 +347,9 @@ class ServiceRecord extends Binder {
                         // If it gave us a garbage notification, it doesn't
                         // get to be foreground.
                         ams.setServiceForeground(name, ServiceRecord.this,
-                                localForegroundId, null, true);
+                                0, null, true);
+                        ams.crashApplication(appUid, appPid, localPackageName,
+                                "Bad notification for startForeground: " + e);
                     }
                 }
             });