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

Commit ce717adc authored by Todd Kennedy's avatar Todd Kennedy
Browse files

Add debugging to diagnose failure

Bug: 186558987
Test: Manual. Builds and runs
Change-Id: Ia65d60590f33f4321697e9e1814b47f5e185bf99
parent f3c059b9
Loading
Loading
Loading
Loading
+21 −10
Original line number Diff line number Diff line
@@ -26019,11 +26019,14 @@ public class PackageManagerService extends IPackageManager.Stub
        @Override
        public String[] getNamesForUids(int[] uids) throws RemoteException {
            String[] names = null;
            String[] results = null;
            try {
                if (uids == null || uids.length == 0) {
                    return null;
                }
            final String[] names = PackageManagerService.this.getNamesForUids(uids);
            final String[] results = (names != null) ? names : new String[uids.length];
                names = PackageManagerService.this.getNamesForUids(uids);
                results = (names != null) ? names : new String[uids.length];
                // massage results so they can be parsed by the native binder
                for (int i = results.length - 1; i >= 0; --i) {
                    if (results[i] == null) {
@@ -26031,6 +26034,14 @@ public class PackageManagerService extends IPackageManager.Stub
                    }
                }
                return results;
            } catch (Throwable t) {
                // STOPSHIP(186558987): revert addition of try/catch/log
                Slog.e(TAG, "uids: " + Arrays.toString(uids));
                Slog.e(TAG, "names: " + Arrays.toString(names));
                Slog.e(TAG, "results: " + Arrays.toString(results));
                Slog.e(TAG, "throwing exception", t);
                throw t;
            }
        }
        // NB: this differentiates between preloads and sideloads