am fbf61b1f: Merge "Fingerprint: Safeguard against empty paths." into mnc-dev

* commit 'fbf61b1f761618af83c82a3a61f2bff81a66fbf8':
  Fingerprint: Safeguard against empty paths.
This commit is contained in:
Sasha Levitskiy 2015-06-23 03:20:24 +00:00 committed by Android Git Automerger
commit 7f48db407e

View file

@ -160,8 +160,8 @@ uint64_t FingerprintDaemonProxy::getAuthenticatorId() {
int32_t FingerprintDaemonProxy::setActiveGroup(int32_t groupId, const uint8_t* path,
ssize_t pathlen) {
if (pathlen >= PATH_MAX) {
ALOGE("Path name is too long\n");
if (pathlen >= PATH_MAX || pathlen <= 0) {
ALOGE("Bad path length: %zd", pathlen);
return -1;
}
// Convert to null-terminated string
@ -170,7 +170,6 @@ int32_t FingerprintDaemonProxy::setActiveGroup(int32_t groupId, const uint8_t* p
path_name[pathlen] = '\0';
ALOG(LOG_VERBOSE, LOG_TAG, "setActiveGroup(%d, %s, %zu)", groupId, path_name, pathlen);
return mDevice->set_active_group(mDevice, groupId, path_name);
return -1;
}
int64_t FingerprintDaemonProxy::openHal() {