From 7d18ceb59f3ba2f975907381c07da62fbdf7b26b Mon Sep 17 00:00:00 2001
From: Tyler Gunn <tgunn@google.com>
Date: Fri, 15 Aug 2014 17:01:19 -0700
Subject: [PATCH] Removing permission check in ConnectionService.

This breaks RemoteConnectionServices, such as hangouts.

Bug: 16171070
Change-Id: I9b53fefe9c3246abb1e9e11109957fb2f8502de8
---
 .../android/telecomm/ConnectionService.java   | 22 -------------------
 1 file changed, 22 deletions(-)

diff --git a/telecomm/java/android/telecomm/ConnectionService.java b/telecomm/java/android/telecomm/ConnectionService.java
index 8ab5e1341ca0d..d2d4828054a6f 100644
--- a/telecomm/java/android/telecomm/ConnectionService.java
+++ b/telecomm/java/android/telecomm/ConnectionService.java
@@ -16,7 +16,6 @@
 
 package android.telecomm;
 
-import android.Manifest;
 import android.annotation.SdkConstant;
 import android.app.PendingIntent;
 import android.app.Service;
@@ -27,8 +26,6 @@ import android.os.Handler;
 import android.os.IBinder;
 import android.os.Looper;
 import android.os.Message;
-import android.os.Parcel;
-import android.os.RemoteException;
 import android.telephony.DisconnectCause;
 
 import com.android.internal.os.SomeArgs;
@@ -83,18 +80,6 @@ public abstract class ConnectionService extends Service {
     private final ConnectionServiceAdapter mAdapter = new ConnectionServiceAdapter();
 
     private final IBinder mBinder = new IConnectionService.Stub() {
-        /**
-         * Enforces the requirement that all calls into the ConnectionService require the
-         * {@code BIND_CONNECTION_SERVICE} permission.
-         */
-        @Override
-        public boolean onTransact(int code, Parcel data, Parcel reply,
-                int flags) throws RemoteException
-        {
-            enforceBindConnectionServicePermission();
-            return super.onTransact(code, data, reply, flags);
-        }
-
         @Override
         public void addConnectionServiceAdapter(IConnectionServiceAdapter adapter) {
             mHandler.obtainMessage(MSG_ADD_CONNECTION_SERVICE_ADAPTER, adapter).sendToTarget();
@@ -844,11 +829,4 @@ public abstract class ConnectionService extends Service {
         Log.w(this, "%s - Cannot find Connection %s", action, callId);
         return Connection.getNullConnection();
     }
-
-    /**
-     * Enforces the {@code BIND_CONNECTION_SERVICE} permission for connection service calls.
-     */
-    private void enforceBindConnectionServicePermission() {
-        enforceCallingPermission(Manifest.permission.BIND_CONNECTION_SERVICE, null);
-    }
 }