Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

Android RemoteExceptions and Services

So I've written a Service and an Activity for the Android OS.

My service is running in it's own process, so all the communication between my Activities and the Service happens via IPC. I use the standard Android .aidl mechanism for this.

So far everything works fine. However, the AIDL generates all method stubs using "throws RemoteException" so I have to handle them.

I did a quick grep on the entire Android source-code and only found three cases where this exception is ever thrown. These are in a different service that I don't connect with.

I checked the C-sources as well because in theory RemoteExceptions can be generated using the JNI interface.. Nothing turned up.

I have the impression that everyone just handles them like this:

  try {      mService.someMethodCall (someArguments);    } catch (RemoteException e) {      e.printStackTrace();    } 

This is not solid code, and I don't want something like this in my code-base.

Besides that: I tried to throw a RemoteException via IPC myself and all I got was a stack-trace and a system log message that tells me that exceptions aren't supported yet. My application never saw the exception and the services that threw the exception ended up in a very strange state (halfway working) :-(

The questions are:

  • Do these exceptions ever get thrown?

  • Has anyone ever seen such a try-catch block catching a RemoteException?

  • Could it be that they don't exist and that we are just forced to deal with them because the "throws RemoteException" is dead code or a left-over inside the AIDL compiler?

Disclamer: I haven't read the entire source-code. I used Grep to find the occurrences of RemoteException, so I may have missed some due to different whitespace usage.

like image 250
Nils Pipenbrinck Avatar asked Jul 01 '10 09:07

Nils Pipenbrinck


People also ask

What is remote exception in Android?

RemoteException is a actually just a base class for other Binder related exceptions and that it is a subclass, android. os. DeadObjectException , which is thrown within the native code of Binder.

What is RemoteException in java?

A RemoteException is the common superclass for a number of communication-related exceptions that may occur during the execution of a remote method call. Each method of a remote interface, an interface that extends java. rmi. Remote , must list RemoteException in its throws clause.

What is Ibinder in Android?

Base interface for a remotable object, the core part of a lightweight remote procedure call mechanism designed for high performance when performing in-process and cross-process calls. This interface describes the abstract protocol for interacting with a remotable object.


1 Answers

These exceptions do indeed get thrown and you should write appropriate try/catch logic to handle the situation where a remote method you invoked on a service did not complete.

As far as your investigation, you were on the right track looking through the native sources. What you may have overlooked is that android.os.RemoteException is a actually just a base class for other Binder related exceptions and that it is a subclass, android.os.DeadObjectException, which is thrown within the native code of Binder.

An activity will see this exception if it makes use of a service running in another process that dies in the middle of performing a request. I was able to prove this to myself by making the following minor changes to Marko Gargenta's AIDLDemo example.

First, make sure the service runs in its own process by updating the AndroidManifest.xml:

<?xml version="1.0" encoding="utf-8"?> <manifest xmlns:android="http://schemas.android.com/apk/res/android"     package="com.marakana" android:versionCode="1" android:versionName="1.0">     <application android:icon="@drawable/icon" android:label="@string/app_name"         android:theme="@android:style/Theme.Light">         <activity android:name=".AIDLDemo" android:label="@string/app_name">             <intent-filter>                 <action android:name="android.intent.action.MAIN" />                 <category android:name="android.intent.category.LAUNCHER" />             </intent-filter>         </activity>         <!--ADD THE android:process TAG TO THE SERVICE-->         <service android:name=".AdditionService" android:process=":process2"/>     </application>     <uses-sdk android:minSdkVersion="3" /> </manifest>  

Then modify the add method to exit prematurely:

@Override public IBinder onBind(Intent intent) {      return new IAdditionService.Stub() {         /**          * Implementation of the add() method          */         public int add(int value1, int value2) throws RemoteException {             Log.d(TAG, String.format("AdditionService.add(%d, %d)", value1,                     value2));                          System.exit(-1); // KILL THE PROCESS BEFORE IT CAN RESPOND                          return value1 + value2;         }      }; } 

In logcat you see the service process die, the activity receive a DeadObjectException, and ultimately the system respawn the service process.

D/AdditionService( 1379): AdditionService.add(1, 1) I/AndroidRuntime( 1379): AndroidRuntime onExit calling exit(-1) D/Zygote  (   32): Process 1379 exited cleanly (255) I/ActivityManager(   58): Process com.marakana:process2 (pid 1379) has died. W/ActivityManager(   58): Scheduling restart of crashed service com.marakana/.AdditionService in 5000ms D/AIDLDemo( 1372): onClick failed with: android.os.DeadObjectException W/System.err( 1372): android.os.DeadObjectException W/System.err( 1372):    at android.os.BinderProxy.transact(Native Method) W/System.err( 1372):    at com.marakana.IAdditionService$Stub$Proxy.add(IAdditionService.java:95) W/System.err( 1372):    at com.marakana.AIDLDemo$1.onClick(AIDLDemo.java:81) W/System.err( 1372):    at android.view.View.performClick(View.java:2408) W/System.err( 1372):    at android.view.View$PerformClick.run(View.java:8816) W/System.err( 1372):    at android.os.Handler.handleCallback(Handler.java:587) W/System.err( 1372):    at android.os.Handler.dispatchMessage(Handler.java:92) W/System.err( 1372):    at android.os.Looper.loop(Looper.java:123) W/System.err( 1372):    at android.app.ActivityThread.main(ActivityThread.java:4627) W/System.err( 1372):    at java.lang.reflect.Method.invokeNative(Native Method) W/System.err( 1372):    at java.lang.reflect.Method.invoke(Method.java:521) W/System.err( 1372):    at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:868) W/System.err( 1372):    at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:626) W/System.err( 1372):    at dalvik.system.NativeStart.main(Native Method) D/AIDLDemo( 1372): onServiceDisconnected() disconnected I/ActivityManager(   58): Start proc com.marakana:process2 for service com.marakana/.AdditionService: pid=1399 uid=10037 gids={1015} D/AdditionService( 1399): onCreate() D/AIDLDemo( 1372): onServiceConnected() connected 

I would imagine if your service was running in the same process as your activity you might never see this exception but then again if that were the case you probably wouldn't be bothering with AIDL.

Additionally, as you discovered, Android does not tunnel exceptions between processes. If you need to communicate an error back to a calling activity then you need to use other means.

like image 54
Tim Kryger Avatar answered Oct 05 '22 18:10

Tim Kryger