diff options
Diffstat (limited to 'fingerprintd/FingerprintDaemonProxy.cpp')
-rw-r--r-- | fingerprintd/FingerprintDaemonProxy.cpp | 5 |
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 | ||
161 | int32_t FingerprintDaemonProxy::setActiveGroup(int32_t groupId, const uint8_t* path, | 161 | int32_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 | ||
176 | int64_t FingerprintDaemonProxy::openHal() { | 175 | int64_t FingerprintDaemonProxy::openHal() { |