Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 6493542c authored by Deva Ramasubramanian's avatar Deva Ramasubramanian Committed by Steve Kondik
Browse files

libstagefright: Fix handling of 'skip' atom in MPEG4Extractor

As per the MPEG4 specs, the contents of the skip atom are irrelevant
and can be ignored. Currently, the parser attempted to parse the skip
atom which doesn't have any structure.

Attempting to parse the skip atom causes a problem if the bytes immediately
following the atom type (which denote the size for most other atoms)
form a extremely large number.  In that case we add this large number to
the offset.  Eventually, we get out of this atom because we can't parse
it.  But we fail the 'sanity check' in the previous recursive level
since '*offset != stop_offset' would not be true.

This change completely ignores the contents of the skip atom.

Change-Id: I837e8279aa1417f7c35ed66be80aa6d313a77722
parent a611ed2b
Loading
Loading
Loading
Loading
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment