OSDN Git Service

Fix failure to check HIDL return status
authorJeff Tinker <jtinker@google.com>
Mon, 7 Aug 2017 22:58:41 +0000 (15:58 -0700)
committerJeff Tinker <jtinker@google.com>
Mon, 7 Aug 2017 23:13:27 +0000 (16:13 -0700)
HIDL transaction status wasn't being checked
resulting in a SIGABRT from createPlugin when the
remote process dies.

bug: 64441780
test: run Play Movies and repeat killing
  android.hardware.drm@1.0-service.widevine, check
  log for transaction failed messages.

Change-Id: I0ea70578d60bb84cf693f158963181781d7e35c3

drm/libmediadrm/DrmHal.cpp

index 2b7987a..074489a 100644 (file)
@@ -245,6 +245,11 @@ sp<IDrmPlugin> DrmHal::makeDrmPlugin(const sp<IDrmFactory>& factory,
                 plugin = hPlugin;
             }
         );
+
+    if (!hResult.isOk()) {
+        ALOGE("createPlugin remote call failed");
+    }
+
     return plugin;
 }