From e1af92e44d432f260b347ad5cf0452b0992dbd5e Mon Sep 17 00:00:00 2001 From: Hall Liu Date: Fri, 5 Jan 2018 12:08:51 -0800 Subject: [PATCH] Add missing return statement in setTempFileRootDirectory Return without setting the local temp file root if the middleware reports a failure. Test: cts Change-Id: I5e8ea1f7941a7157d88536925f681044d26ef540 --- telephony/java/android/telephony/MbmsDownloadSession.java | 1 + 1 file changed, 1 insertion(+) diff --git a/telephony/java/android/telephony/MbmsDownloadSession.java b/telephony/java/android/telephony/MbmsDownloadSession.java index 43d499a4269d4..da3d87bc72114 100644 --- a/telephony/java/android/telephony/MbmsDownloadSession.java +++ b/telephony/java/android/telephony/MbmsDownloadSession.java @@ -437,6 +437,7 @@ public class MbmsDownloadSession implements AutoCloseable { int result = downloadService.setTempFileRootDirectory(mSubscriptionId, filePath); if (result != MbmsErrors.SUCCESS) { sendErrorToApp(result, null); + return; } } catch (RemoteException e) { mService.set(null);