Merge "NativeLibraryHelper: Minor fixes"
am: c1536f0d10
Change-Id: I81e69315c5f74341ddf3472263462f8fdf935797
This commit is contained in:
@@ -243,7 +243,7 @@ copyFileIfChanged(JNIEnv *env, void* arg, ZipFileRO* zipFile, ZipEntryRO zipEntr
|
|||||||
return INSTALL_FAILED_INTERNAL_ERROR;
|
return INSTALL_FAILED_INTERNAL_ERROR;
|
||||||
}
|
}
|
||||||
|
|
||||||
*(localFileName + nativeLibPath.size()) = '/';
|
*(localTmpFileName + nativeLibPath.size()) = '/';
|
||||||
|
|
||||||
if (strlcpy(localTmpFileName + nativeLibPath.size(), TMP_FILE_PATTERN,
|
if (strlcpy(localTmpFileName + nativeLibPath.size(), TMP_FILE_PATTERN,
|
||||||
TMP_FILE_PATTERN_LEN - nativeLibPath.size()) != TMP_FILE_PATTERN_LEN) {
|
TMP_FILE_PATTERN_LEN - nativeLibPath.size()) != TMP_FILE_PATTERN_LEN) {
|
||||||
@@ -344,6 +344,11 @@ public:
|
|||||||
const char* lastSlash = strrchr(fileName, '/');
|
const char* lastSlash = strrchr(fileName, '/');
|
||||||
ALOG_ASSERT(lastSlash != NULL, "last slash was null somehow for %s\n", fileName);
|
ALOG_ASSERT(lastSlash != NULL, "last slash was null somehow for %s\n", fileName);
|
||||||
|
|
||||||
|
// Skip directories.
|
||||||
|
if (*(lastSlash + 1) == 0) {
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
|
||||||
// Make sure the filename is safe.
|
// Make sure the filename is safe.
|
||||||
if (!isFilenameSafe(lastSlash + 1)) {
|
if (!isFilenameSafe(lastSlash + 1)) {
|
||||||
continue;
|
continue;
|
||||||
|
|||||||
Reference in New Issue
Block a user