commit b0cea9ce242f0914691f052215fe73a510c0ed18
author James Zern <jzern@google.com> Mon May 01 15:17:30 2017 -0700
committer James Zern <jzern@google.com> Mon May 01 15:17:30 2017 -0700
tree 4cfa0a6708dc11ded29ea8c751a21fd0258abff6
parent a65b9e02874c0e8526cef9b0f59b72eb9b2132d4

Add(Audio|Video)Track: fix memory leak on failure

quiets a static analysis warning

Change-Id: Iaa6909e94deae81ace3e65f37ab746d72031620f

mkvmuxer/mkvmuxer.cc

diff --git a/mkvmuxer/mkvmuxer.cc b/mkvmuxer/mkvmuxer.cc
index b7ab131..9bf1924 100644
--- a/mkvmuxer/mkvmuxer.cc
+++ b/mkvmuxer/mkvmuxer.cc
@@ -3361,7 +3361,10 @@
   track->set_width(width);
   track->set_height(height);
 
-  tracks_.AddTrack(track, number);
+  if (!tracks_.AddTrack(track, number)) {
+    delete track;
+    return 0;
+  }
   has_video_ = true;
 
   return track->number();
@@ -3401,7 +3404,10 @@
   track->set_sample_rate(sample_rate);
   track->set_channels(channels);
 
-  tracks_.AddTrack(track, number);
+  if (!tracks_.AddTrack(track, number)) {
+    delete track;
+    return 0;
+  }
 
   return track->number();
 }