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

Commit 9dd966fb authored by Wu Ahan's avatar Wu Ahan Committed by Automerger Merge Worker
Browse files

Merge "Decode the input of both setStream and setResource calls first" into... 

Merge "Decode the input of both setStream and setResource calls first" into sc-v2-dev am: c262621e am: 360a31df

Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/16375868

Change-Id: Ib4fae153fa420ad892c60f55bff247e1d3a618ae
parents 9e1398c2 360a31df

core/java/android/app/WallpaperManager.java

+29 −11
Original line number Diff line number Diff line
@@ -1489,18 +1489,27 @@ public class WallpaperManager { 
                    mContext.getUserId());

            if (fd != null) {

                FileOutputStream fos = null;

                boolean ok = false;

                final Bitmap tmp = BitmapFactory.decodeStream(resources.openRawResource(resid));

                try {

                    // If the stream can't be decoded, treat it as an invalid input.

                    if (tmp != null) {

                        fos = new ParcelFileDescriptor.AutoCloseOutputStream(fd);

                    copyStreamToWallpaperFile(resources.openRawResource(resid), fos);

                        tmp.compress(Bitmap.CompressFormat.PNG, 100, fos);

                        // The 'close()' is the trigger for any server-side image manipulation,

                        // so we must do that before waiting for completion.

                        fos.close();

                        completion.waitForCompletion();

                    } else {

                        throw new IllegalArgumentException(

                                "Resource 0x" + Integer.toHexString(resid) + " is invalid");

                    }

                } finally {

                    // Might be redundant but completion shouldn't wait unless the write

                    // succeeded; this is a fallback if it threw past the close+wait.

                    IoUtils.closeQuietly(fos);

                    if (tmp != null) {

                        tmp.recycle();

                    }

                }

            }

        } catch (RemoteException e) {
@@ -1742,13 +1751,22 @@ public class WallpaperManager { 
                    result, which, completion, mContext.getUserId());

            if (fd != null) {

                FileOutputStream fos = null;

                final Bitmap tmp = BitmapFactory.decodeStream(bitmapData);

                try {

                    // If the stream can't be decoded, treat it as an invalid input.

                    if (tmp != null) {

                        fos = new ParcelFileDescriptor.AutoCloseOutputStream(fd);

                    copyStreamToWallpaperFile(bitmapData, fos);

                        tmp.compress(Bitmap.CompressFormat.PNG, 100, fos);

                        fos.close();

                        completion.waitForCompletion();

                    } else {

                        throw new IllegalArgumentException("InputStream is invalid");

                    }

                } finally {

                    IoUtils.closeQuietly(fos);

                    if (tmp != null) {

                        tmp.recycle();

                    }

                }

            }

        } catch (RemoteException e) {