#1376 fix: use new key event relay service AIDL definition for unregistering a specific callback

Author: sds100

app/src/main/aidl/io/github/sds100/keymapper/api/IKeyEventRelayService.aidl

@@ -18,9 +18,12 @@ interface IKeyEventRelayService {
     void registerCallback(IKeyEventRelayServiceCallback client);
 
     /**
-     * Unregister a callback to receive key events from this relay service. The service
-     * checks the process uid of the caller to this method and only permits certain applications
-     * from connecting.
+     * Unregister all the callbacks associated with the calling package from this relay service.
+     */
+    void unregisterAllCallbacks();
+
+    /**
+     * Unregister a callback to receive key events from this relay service.
      */
-    void unregisterCallback();
+    void unregisterCallback(IKeyEventRelayServiceCallback client);
 }

app/src/main/java/io/github/sds100/keymapper/inputmethod/latin/KeyEventRelayServiceWrapper.kt

@@ -128,6 +128,9 @@ class KeyEventRelayServiceWrapperImpl(
             }
         } catch (e: SecurityException) {
             Log.e(LatinIME.TAG, e.toString())
+
+            // Docs say to unbind if there is a security exception.
+            ctx.unbindService(serviceConnection)
         }
     }
 
@@ -138,7 +141,7 @@ class KeyEventRelayServiceWrapperImpl(
         // because the connection is already broken at that point and it
         // will fail.
         try {
-            keyEventRelayService?.unregisterCallback()
+            keyEventRelayService?.unregisterCallback(callback)
             ctx.unbindService(serviceConnection)
         } catch (e: RemoteException) {
             // do nothing