• Andreas Rheinhardt's avatar
    cbs_mpeg2: Fix parsing of picture and slice headers · d9182f04
    Andreas Rheinhardt authored
    1. The extra information in slice headers was parsed incorrectly:
    In the first reading pass to derive the length of the extra information,
    one should look at bits n, n + 9, n + 18, ... and check whether they
    equal one (further extra information) or zero (end of extra information),
    but instead bits n, n + 8, n + 16, ... were inspected. The second pass
    of reading (where the length is already known and the bytes between the
    length-determining bits are copied into a buffer) did not record what
    was in bits n, n + 9, n + 18, ..., presuming they equal one. And during
    writing, the bytes in the buffer are interleaved with set bits and
    written. This means that if the detected length of the extra information
    was greater than the real length, the output was corrupted. Fortunately
    no sample is known that made use of this mechanism: The extra information
    in slices is still marked as reserved in the specifications. cbs_mpeg2
    is now ready in case this changes.
    
    2. Furthermore, the buffer is now padded and slightly different, but
    very similar code for reading resp. writing has been replaced by code
    used for both. This was made possible by a new macro, the equivalent
    to cbs_h2645's fixed().
    
    3. These changes also made it possible to remove the extra_bit_slice
    element from the MPEG2RawSliceHeader structure. Said element was always
    zero except when the detected length of the extra information was less
    than the real length.
    
    4. The extra information in picture headers (which uses essentially the
    same syntax as the extra information in slice headers) has simply been
    forgotten. This meant that if this extra information was present, it was
    discarded during reading; and unfortunately writing created invalid
    bitstreams in this case (an extra_bit_picture - the last set bit of the
    whole unit - indicated that there would be a further byte of data,
    although the output didn't contain said data).
    
    This has been fixed; both types of extra information are now parsed via
    the same code and essentially passed through.
    Signed-off-by: 's avatarAndreas Rheinhardt <andreas.rheinhardt@gmail.com>
    d9182f04
Name
Last commit
Last update
compat Loading commit data...
doc Loading commit data...
ffbuild Loading commit data...
fftools Loading commit data...
libavcodec Loading commit data...
libavdevice Loading commit data...
libavfilter Loading commit data...
libavformat Loading commit data...
libavresample Loading commit data...
libavutil Loading commit data...
libpostproc Loading commit data...
libswresample Loading commit data...
libswscale Loading commit data...
presets Loading commit data...
tests Loading commit data...
tools Loading commit data...
.gitattributes Loading commit data...
.gitignore Loading commit data...
.travis.yml Loading commit data...
CONTRIBUTING.md Loading commit data...
COPYING.GPLv2 Loading commit data...
COPYING.GPLv3 Loading commit data...
COPYING.LGPLv2.1 Loading commit data...
COPYING.LGPLv3 Loading commit data...
CREDITS Loading commit data...
Changelog Loading commit data...
INSTALL.md Loading commit data...
LICENSE.md Loading commit data...
MAINTAINERS Loading commit data...
Makefile Loading commit data...
README.md Loading commit data...
RELEASE Loading commit data...
configure Loading commit data...