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

Commit de57d0d0 authored by Hall Liu's avatar Hall Liu Committed by Gerrit Code Review
Browse files

Merge "Add additional logging for EMBMS edge cases"

parents 18e47b61 427e43eb
Loading
Loading
Loading
Loading
+2 −2
Original line number Diff line number Diff line
@@ -920,11 +920,11 @@ public class MbmsDownloadSession implements AutoCloseable {
        try {
            if (!token.createNewFile()) {
                throw new RuntimeException("Failed to create download token for request "
                        + request);
                        + request + ". Token location is " + token.getPath());
            }
        } catch (IOException e) {
            throw new RuntimeException("Failed to create download token for request " + request
                    + " due to IOException " + e);
                    + " due to IOException " + e + ". Attempted to write to " + token.getPath());
        }
    }

+5 −1
Original line number Diff line number Diff line
@@ -297,7 +297,9 @@ public class MbmsDownloadReceiver extends BroadcastReceiver {
        for (Uri tempFileUri : tempFiles) {
            if (verifyTempFilePath(context, request.getFileServiceId(), tempFileUri)) {
                File tempFile = new File(tempFileUri.getSchemeSpecificPart());
                tempFile.delete();
                if (!tempFile.delete()) {
                    Log.w(LOG_TAG, "Failed to delete temp file at " + tempFile.getPath());
                }
            }
        }
    }
@@ -474,6 +476,8 @@ public class MbmsDownloadReceiver extends BroadcastReceiver {

        if (!MbmsUtils.isContainedIn(
                MbmsUtils.getEmbmsTempFileDirForService(context, serviceId), tempFile)) {
            Log.w(LOG_TAG, "File at " + path + " is not contained in the temp file root," +
                    " which is " + MbmsUtils.getEmbmsTempFileDirForService(context, serviceId));
            return false;
        }