Skip to content
Permalink
Browse files

Defensive check against reported crash in MatroskaImport::~MatroskaIm…

…port

Unsure how it could happen by reading code, but it's possible for el_l1 to
be an invalid pointer there if an exception is thrown because the return value
won't be set.

git-svn-id: https://svn.perian.org/trunk@1517 621663c8-3916-0410-8f58-edc14a8543d5
  • Loading branch information...
astrange
astrange committed Jul 25, 2012
1 parent 5ffb164 commit 5d31ed40fa5d5ef1023437019a69fadbb24b2a5f
Showing with 1 addition and 0 deletions.
  1. +1 −0 MatroskaImportPrivate.cpp
@@ -180,6 +180,7 @@ EbmlElement * MatroskaImport::NextLevel1Element()
if (el_l1) {
el_l1->SkipData(*aStream, el_l1->Generic().Context);
delete el_l1;
el_l1 = NULL;
}

el_l1 = aStream->FindNextElement(el_l0->Generic().Context, upperLevel, 0xFFFFFFFFL, true);

0 comments on commit 5d31ed4

Please sign in to comment.
You can’t perform that action at this time.