commit b2a7ed2e9d10d3aa277b0457a2601c99217e273f
author Marco Nelissen <marcone@google.com> Fri Jan 18 10:34:03 2019 -0800
committer Marco Nelissen <marcone@google.com> Fri Jan 18 10:34:03 2019 -0800
tree b2775d1fca12ead36f594ab516215baf6e0856e4
parent 0e7fbcb161e2e806ffb2f321cd14e92924463ec2

Fix test failures

Bug: 123077756
Test: CTS
Change-Id: I2b440fef382bd7a6b5d956defd7715ec614f8d31

include/media/MediaExtractorPluginHelper.h

diff --git a/include/media/MediaExtractorPluginHelper.h b/include/media/MediaExtractorPluginHelper.h
index f4d4da6..b86f177 100644
--- a/include/media/MediaExtractorPluginHelper.h
+++ b/include/media/MediaExtractorPluginHelper.h
@@ -171,6 +171,9 @@
 };
 
 inline CMediaTrack *wrap(MediaTrackHelper *track) {
+    if (track == nullptr) {
+        return nullptr;
+    }
     CMediaTrack *wrapper = (CMediaTrack*) malloc(sizeof(CMediaTrack));
     wrapper->data = track;
     wrapper->free = [](void *data) -> void {

include/media/MediaTrack.h

diff --git a/include/media/MediaTrack.h b/include/media/MediaTrack.h
index e828a7f..493eba3 100644
--- a/include/media/MediaTrack.h
+++ b/include/media/MediaTrack.h
@@ -142,7 +142,7 @@
 
 class MediaTrackCUnwrapper : public MediaTrack {
 public:
-    explicit MediaTrackCUnwrapper(CMediaTrack *wrapper);
+    static MediaTrackCUnwrapper *create(CMediaTrack *wrapper);
 
     virtual status_t start();
     virtual status_t stop();
@@ -155,6 +155,7 @@
     virtual ~MediaTrackCUnwrapper();
 
 private:
+    explicit MediaTrackCUnwrapper(CMediaTrack *wrapper);
     CMediaTrack *wrapper;
     MediaBufferGroup *bufferGroup;
 };