From 80b10b5fc1b7f85909271eb3aae3d3844980aa8f Mon Sep 17 00:00:00 2001 From: eguven Date: Wed, 8 Jun 2016 03:14:51 -0700 Subject: [PATCH] TestUtil.consumeTestData(): Make sure extractor doesn't use the position holder out of purpose. ------------- Created by MOE: https://github.com/google/moe MOE_MIGRATED_REVID=124335837 --- .../java/com/google/android/exoplayer/testutil/TestUtil.java | 3 +++ 1 file changed, 3 insertions(+) diff --git a/testutils/src/main/java/com/google/android/exoplayer/testutil/TestUtil.java b/testutils/src/main/java/com/google/android/exoplayer/testutil/TestUtil.java index b29f9d6ac5..08da810460 100644 --- a/testutils/src/main/java/com/google/android/exoplayer/testutil/TestUtil.java +++ b/testutils/src/main/java/com/google/android/exoplayer/testutil/TestUtil.java @@ -84,6 +84,9 @@ public class TestUtil { int readResult = Extractor.RESULT_CONTINUE; while (readResult != Extractor.RESULT_END_OF_INPUT) { try { + // Extractor.read should not read seekPositionHolder.position. Set it to a value that's + // likely to cause test failure if a read does occur. + seekPositionHolder.position = Long.MIN_VALUE; readResult = extractor.read(input, seekPositionHolder); if (readResult == Extractor.RESULT_SEEK) { long seekPosition = seekPositionHolder.position;