summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
Diffstat (limited to 'fingerprintd/FingerprintDaemonProxy.cpp')
-rw-r--r--fingerprintd/FingerprintDaemonProxy.cpp5
1 files changed, 2 insertions, 3 deletions
diff --git a/fingerprintd/FingerprintDaemonProxy.cpp b/fingerprintd/FingerprintDaemonProxy.cpp
index a55f30a2d..4c8e47850 100644
--- a/fingerprintd/FingerprintDaemonProxy.cpp
+++ b/fingerprintd/FingerprintDaemonProxy.cpp
@@ -160,8 +160,8 @@ uint64_t FingerprintDaemonProxy::getAuthenticatorId() {
160 160
161int32_t FingerprintDaemonProxy::setActiveGroup(int32_t groupId, const uint8_t* path, 161int32_t FingerprintDaemonProxy::setActiveGroup(int32_t groupId, const uint8_t* path,
162 ssize_t pathlen) { 162 ssize_t pathlen) {
163 if (pathlen >= PATH_MAX) { 163 if (pathlen >= PATH_MAX || pathlen <= 0) {
164 ALOGE("Path name is too long\n"); 164 ALOGE("Bad path length: %zd", pathlen);
165 return -1; 165 return -1;
166 } 166 }
167 // Convert to null-terminated string 167 // Convert to null-terminated string
@@ -170,7 +170,6 @@ int32_t FingerprintDaemonProxy::setActiveGroup(int32_t groupId, const uint8_t* p
170 path_name[pathlen] = '\0'; 170 path_name[pathlen] = '\0';
171 ALOG(LOG_VERBOSE, LOG_TAG, "setActiveGroup(%d, %s, %zu)", groupId, path_name, pathlen); 171 ALOG(LOG_VERBOSE, LOG_TAG, "setActiveGroup(%d, %s, %zu)", groupId, path_name, pathlen);
172 return mDevice->set_active_group(mDevice, groupId, path_name); 172 return mDevice->set_active_group(mDevice, groupId, path_name);
173 return -1;
174} 173}
175 174
176int64_t FingerprintDaemonProxy::openHal() { 175int64_t FingerprintDaemonProxy::openHal() {