BPackageManager::_CommitPackageChanges(): Fix error handling
The error returned by CommitTransaction() is just indicates whether transactionResult has been initialized.
This commit is contained in:
@@ -518,7 +518,9 @@ BPackageManager::_CommitPackageChanges(Transaction& transaction)
|
|||||||
BDaemonClient::BCommitTransactionResult transactionResult;
|
BDaemonClient::BCommitTransactionResult transactionResult;
|
||||||
status_t error = fInstallationInterface->CommitTransaction(transaction,
|
status_t error = fInstallationInterface->CommitTransaction(transaction,
|
||||||
transactionResult);
|
transactionResult);
|
||||||
if (error != B_OK) {
|
if (error != B_OK)
|
||||||
|
DIE(error, "failed to commit transaction");
|
||||||
|
if (transactionResult.Error() != B_OK) {
|
||||||
DIE("failed to commit transaction: %s",
|
DIE("failed to commit transaction: %s",
|
||||||
transactionResult.FullErrorMessage().String());
|
transactionResult.FullErrorMessage().String());
|
||||||
}
|
}
|
||||||
|
|||||||
Reference in New Issue
Block a user