Enable >= 2GiB file size support in backupToTar for 32 bit targets.

backupToTar function used in fullbackup currently uses off_t to
represent file sizes which overflows for files having size >= 2 GiB
on 32 bit target architectures.

This CL replaces off_t with off64_t to overcome this limitation.

Fixes: 157633626
Test: atest libandroidfw_tests
Test: Locally running a backup with a test app having a file of size
2GiB and verifying the size variable in backupToTar doesn't overflow
on a 32 bit target.

Change-Id: I404cced810f331ecb9327495ce1f092f1fa2d378
This commit is contained in:
Abhinav Aggarwal
2020-06-02 13:47:35 +01:00
parent 8f278fb31e
commit 0c36a942c9
5 changed files with 68 additions and 4 deletions

View File

@@ -111,10 +111,10 @@ static jint backupToTar(JNIEnv* env, jobject clazz, jstring packageNameObj,
return (jint) -1;
}
off_t tarSize = 0;
off64_t tarSize = 0;
jint err = write_tarfile(packageName, domain, rootpath, path, &tarSize, writer);
if (!err) {
//ALOGI("measured [%s] at %lld", path.string(), (long long) tarSize);
ALOGI("measured [%s] at %lld", path.string(), (long long)tarSize);
env->CallVoidMethod(dataOutputObj, sFullBackupDataOutput.addSize, (jlong) tarSize);
}

View File

@@ -155,6 +155,7 @@ cc_test {
android: {
srcs: [
"tests/BackupData_test.cpp",
"tests/BackupHelpers_test.cpp",
"tests/ObbFile_test.cpp",
"tests/PosixUtils_test.cpp",
],

View File

@@ -479,7 +479,7 @@ void send_tarfile_chunk(BackupDataWriter* writer, const char* buffer, size_t siz
}
int write_tarfile(const String8& packageName, const String8& domain,
const String8& rootpath, const String8& filepath, off_t* outSize,
const String8& rootpath, const String8& filepath, off64_t* outSize,
BackupDataWriter* writer)
{
// In the output stream everything is stored relative to the root

View File

@@ -137,7 +137,7 @@ int back_up_files(int oldSnapshotFD, BackupDataWriter* dataStream, int newSnapsh
char const* const* files, char const* const *keys, int fileCount);
int write_tarfile(const String8& packageName, const String8& domain,
const String8& rootPath, const String8& filePath, off_t* outSize,
const String8& rootPath, const String8& filePath, off64_t* outSize,
BackupDataWriter* outputStream);
class RestoreHelperBase

View File

@@ -0,0 +1,63 @@
/*
* Copyright (C) 2020 The Android Open Source Project
*
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
* You may obtain a copy of the License at
*
* http://www.apache.org/licenses/LICENSE-2.0
*
* Unless required by applicable law or agreed to in writing, software
* distributed under the License is distributed on an "AS IS" BASIS,
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
* See the License for the specific language governing permissions and
* limitations under the License.
*/
#define LOG_TAG "BackupHelpers_test"
#include <androidfw/BackupHelpers.h>
#include <gtest/gtest.h>
#include <fcntl.h>
#include <utils/String8.h>
#include <android-base/file.h>
namespace android {
class BackupHelpersTest : public testing::Test {
protected:
virtual void SetUp() {
}
virtual void TearDown() {
}
};
TEST_F(BackupHelpersTest, WriteTarFileWithSizeLessThan2GB) {
TemporaryFile tf;
// Allocate a 1 KB file.
off64_t fileSize = 1024;
ASSERT_EQ(0, posix_fallocate64(tf.fd, 0, fileSize));
off64_t tarSize = 0;
int err = write_tarfile(/* packageName */ String8("test-pkg"), /* domain */ String8(""), /* rootpath */ String8(""), /* filePath */ String8(tf.path), /* outSize */ &tarSize, /* writer */ NULL);
ASSERT_EQ(err, 0);
// Returned tarSize includes 512 B for the header.
off64_t expectedTarSize = fileSize + 512;
ASSERT_EQ(tarSize, expectedTarSize);
}
TEST_F(BackupHelpersTest, WriteTarFileWithSizeGreaterThan2GB) {
TemporaryFile tf;
// Allocate a 2 GB file.
off64_t fileSize = 2ll * 1024ll * 1024ll * 1024ll + 512ll;
ASSERT_EQ(0, posix_fallocate64(tf.fd, 0, fileSize));
off64_t tarSize = 0;
int err = write_tarfile(/* packageName */ String8("test-pkg"), /* domain */ String8(""), /* rootpath */ String8(""), /* filePath */ String8(tf.path), /* outSize */ &tarSize, /* writer */ NULL);
ASSERT_EQ(err, 0);
// Returned tarSize includes 512 B for the header.
off64_t expectedTarSize = fileSize + 512;
ASSERT_EQ(tarSize, expectedTarSize);
}
}