commit a5d28736f490910d086dcb75b06bad89161d910a
author Chih-Hung Hsieh <chh@google.com> Wed Nov 29 00:02:41 2017 +0000
committer android-build-merger <android-build-merger@google.com> Wed Nov 29 00:02:41 2017 +0000
tree aa64313c230b3d96a057a95bdff182afe6b74122
parent f17049b9f7eb2cf44aaa8ef7faa1582eb8a7ceb6
parent 5631258f073da42105ad925690684d74b568c9c3

Use -Werror in hardware/interfaces/biometrics/fingerprint
am: 5631258f07

Change-Id: I6c81b78c4d00faccefdaa3e40ebcecc26222c5e9

biometrics/fingerprint/2.1/default/Android.bp

diff --git a/biometrics/fingerprint/2.1/default/Android.bp b/biometrics/fingerprint/2.1/default/Android.bp
index e1d2cf7..b12ce61 100644
--- a/biometrics/fingerprint/2.1/default/Android.bp
+++ b/biometrics/fingerprint/2.1/default/Android.bp
@@ -1,5 +1,6 @@
 cc_binary {
     name: "android.hardware.biometrics.fingerprint@2.1-service",
+    defaults: ["hidl_defaults"],
     init_rc: ["android.hardware.biometrics.fingerprint@2.1-service.rc"],
     vendor: true,
     relative_install_path: "hw",

biometrics/fingerprint/2.1/default/service.cpp

diff --git a/biometrics/fingerprint/2.1/default/service.cpp b/biometrics/fingerprint/2.1/default/service.cpp
index 1697c07..edfaac4 100644
--- a/biometrics/fingerprint/2.1/default/service.cpp
+++ b/biometrics/fingerprint/2.1/default/service.cpp
@@ -35,7 +35,9 @@
     configureRpcThreadpool(1, true /*callerWillJoin*/);
 
     if (bio != nullptr) {
-        bio->registerAsService();
+        if (::android::OK != bio->registerAsService()) {
+            return 1;
+        }
     } else {
         ALOGE("Can't create instance of BiometricsFingerprint, nullptr");
     }