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

Commit 3b2fe9f7 authored by Tao Bao's avatar Tao Bao Committed by Gerrit Code Review
Browse files

Merge "RecoverySystem: Fix the issue in installPackage()."

parents ba17ea75 9f7a0acd
Loading
Loading
Loading
Loading
+1 −1
Original line number Diff line number Diff line
@@ -25,5 +25,5 @@ interface IRecoverySystem {
    boolean uncrypt(in String packageFile, IRecoverySystemProgressListener listener);
    boolean setupBcb(in String command);
    boolean clearBcb();
    void rebootRecoveryWithCommand(in String command);
    void rebootRecoveryWithCommand(in String command, in boolean update);
}
+5 −10
Original line number Diff line number Diff line
@@ -491,15 +491,10 @@ public class RecoverySystem {
                command += securityArg;
            }

            // RECOVERY_SERVICE writes to BCB (bootloader control block) and triggers the reboot.
            RecoverySystem rs = (RecoverySystem) context.getSystemService(
                    Context.RECOVERY_SERVICE);
            if (!rs.setupBcb(command)) {
                throw new IOException("Setup BCB failed");
            }

            // Having set up the BCB (bootloader control block), go ahead and reboot
            PowerManager pm = (PowerManager) context.getSystemService(Context.POWER_SERVICE);
            pm.reboot(PowerManager.REBOOT_RECOVERY_UPDATE);
            rs.rebootRecoveryWithCommand(command, true /* update */);

            throw new IOException("Reboot failed (no permissions?)");
        }
@@ -713,7 +708,7 @@ public class RecoverySystem {
        // Write the command into BCB (bootloader control block) and boot from
        // there. Will not return unless failed.
        RecoverySystem rs = (RecoverySystem) context.getSystemService(Context.RECOVERY_SERVICE);
        rs.rebootRecoveryWithCommand(command.toString());
        rs.rebootRecoveryWithCommand(command.toString(), false);

        throw new IOException("Reboot failed (no permissions?)");
    }
@@ -913,9 +908,9 @@ public class RecoverySystem {
     * Talks to RecoverySystemService via Binder to set up the BCB command and
     * reboot into recovery accordingly.
     */
    private void rebootRecoveryWithCommand(String command) {
    private void rebootRecoveryWithCommand(String command, boolean update) {
        try {
            mService.rebootRecoveryWithCommand(command);
            mService.rebootRecoveryWithCommand(command, update);
        } catch (RemoteException ignored) {
        }
    }
+5 −2
Original line number Diff line number Diff line
@@ -181,7 +181,7 @@ public final class RecoverySystemService extends SystemService {
        }

        @Override // Binder call
        public void rebootRecoveryWithCommand(String command) {
        public void rebootRecoveryWithCommand(String command, boolean update) {
            if (DEBUG) Slog.d(TAG, "rebootRecoveryWithCommand: [" + command + "]");
            synchronized (sRequestLock) {
                if (!setupOrClearBcb(true, command)) {
@@ -190,7 +190,10 @@ public final class RecoverySystemService extends SystemService {

                // Having set up the BCB, go ahead and reboot.
                PowerManager pm = (PowerManager) mContext.getSystemService(Context.POWER_SERVICE);
                pm.reboot(PowerManager.REBOOT_RECOVERY);
                // PowerManagerService may additionally request uncrypting the package when it's
                // to install an update (REBOOT_RECOVERY_UPDATE).
                pm.reboot(update ? PowerManager.REBOOT_RECOVERY_UPDATE :
                        PowerManager.REBOOT_RECOVERY);
            }
        }