Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit cd00a8b3 authored by Ethan Yonker's avatar Ethan Yonker
Browse files

backup: fix return handling when no digest is being created 

If the user selects to skip digest creation, Backup_Partition always returned
false. This patch fixes this problem and somewhat cleans up the error
handling.

Change-Id: I1db0e285cd5ed2bd93756cd27c6f56b8415ffa86
parent da056fdc

partitionmanager.cpp

+14 −22
Original line number Original line Diff line number Diff line
@@ -529,14 +529,12 @@ bool TWPartitionManager::Backup_Partition(PartitionSettings *part_settings) { 
	time(&start);

	time(&start);





	if (part_settings->Part->Backup(part_settings, &tar_fork_pid)) {

	if (part_settings->Part->Backup(part_settings, &tar_fork_pid)) {

		bool digestSuccess = false;

		sync();

		sync();

		string Full_Filename = part_settings->Backup_Folder + "/" + part_settings->Part->Backup_FileName;

		string Full_Filename = part_settings->Backup_Folder + "/" + part_settings->Part->Backup_FileName;

		if (part_settings->generate_digest) {

		if (!part_settings->adbbackup && part_settings->generate_digest) {



			if (!twrpDigestDriver::Make_Digest(Full_Filename))

			if (part_settings->adbbackup)

				goto backup_error;

				digestSuccess = true;

			else

				digestSuccess = twrpDigestDriver::Make_Digest(Full_Filename);

		}

		}





		if (part_settings->Part->Has_SubPartition) {

		if (part_settings->Part->Has_SubPartition) {
@@ -547,18 +545,13 @@ bool TWPartitionManager::Backup_Partition(PartitionSettings *part_settings) { 
				if ((*subpart)->Can_Be_Backed_Up && (*subpart)->Is_SubPartition && (*subpart)->SubPartition_Of == parentPart->Mount_Point) {

				if ((*subpart)->Can_Be_Backed_Up && (*subpart)->Is_SubPartition && (*subpart)->SubPartition_Of == parentPart->Mount_Point) {

					part_settings->Part = *subpart;

					part_settings->Part = *subpart;

					if (!(*subpart)->Backup(part_settings, &tar_fork_pid)) {

					if (!(*subpart)->Backup(part_settings, &tar_fork_pid)) {

						Clean_Backup_Folder(part_settings->Backup_Folder);

						goto backup_error;

						TWFunc::copy_file("/tmp/recovery.log", backup_log, 0644);

						tw_set_default_metadata(backup_log.c_str());

						TWFunc::SetPerformanceMode(false);

						return false;

					}

					}

					sync();

					sync();

					sync();

					sync();

					if (!part_settings->adbbackup && part_settings->generate_digest) {

					if (!part_settings->adbbackup && part_settings->generate_digest) {

						if (!twrpDigestDriver::Make_Digest(Full_Filename)) {

						if (!twrpDigestDriver::Make_Digest(Full_Filename)) {

							TWFunc::SetPerformanceMode(false);

							goto backup_error;

							return false;

						}

						}

					}

					}

				}

				}
@@ -577,16 +570,15 @@ bool TWPartitionManager::Backup_Partition(PartitionSettings *part_settings) { 
		}

		}





		TWFunc::SetPerformanceMode(false);

		TWFunc::SetPerformanceMode(false);

		return digestSuccess;

		return true;

	} else {

	}

backup_error:

	Clean_Backup_Folder(part_settings->Backup_Folder);

	Clean_Backup_Folder(part_settings->Backup_Folder);

	TWFunc::copy_file("/tmp/recovery.log", backup_log, 0644);

	TWFunc::copy_file("/tmp/recovery.log", backup_log, 0644);

	tw_set_default_metadata(backup_log.c_str());

	tw_set_default_metadata(backup_log.c_str());

	TWFunc::SetPerformanceMode(false);

	TWFunc::SetPerformanceMode(false);

	return false;

	return false;

}

}

	return 0;

}





void TWPartitionManager::Clean_Backup_Folder(string Backup_Folder) {

void TWPartitionManager::Clean_Backup_Folder(string Backup_Folder) {

	DIR *d = opendir(Backup_Folder.c_str());

	DIR *d = opendir(Backup_Folder.c_str());