MediaMetadataRetrieverTest: add new test to handle overflowed padding.
Bug: 24623447
Change-Id: Ida117b2386199083a938ef0e24392c849f6b52ea
diff --git a/tests/tests/media/res/raw/id3v2_3_extended_header_overflow_padding.mp3 b/tests/tests/media/res/raw/id3v2_3_extended_header_overflow_padding.mp3
new file mode 100644
index 0000000..7e8d38c
--- /dev/null
+++ b/tests/tests/media/res/raw/id3v2_3_extended_header_overflow_padding.mp3
Binary files differ
diff --git a/tests/tests/media/src/android/media/cts/MediaMetadataRetrieverTest.java b/tests/tests/media/src/android/media/cts/MediaMetadataRetrieverTest.java
index 54e6ef1..ddb717c 100644
--- a/tests/tests/media/src/android/media/cts/MediaMetadataRetrieverTest.java
+++ b/tests/tests/media/src/android/media/cts/MediaMetadataRetrieverTest.java
@@ -26,8 +26,11 @@
import android.test.AndroidTestCase;
import java.io.IOException;
+import android.util.Log;
public class MediaMetadataRetrieverTest extends AndroidTestCase {
+ private static final String LOG_TAG = "MediaMetadataRetrieverTest";
+
protected Resources mResources;
protected MediaMetadataRetriever mRetriever;
@@ -116,6 +119,13 @@
mRetriever.extractMetadata(MediaMetadataRetriever.METADATA_KEY_WRITER));
}
+ public void testID3v2EmbeddedPicture() {
+ setDataSourceFd(R.raw.id3v2_3_extended_header_overflow_padding);
+
+ assertEquals("EmbeddedPicture was other than expected null array",
+ null, mRetriever.getEmbeddedPicture());
+ }
+
public void testSetDataSourceNullPath() {
try {
mRetriever.setDataSource((String)null);
