Signed-off-by: NAlex Converse <alex.converse@gmail.com>

This should be moved to tablegen at some point.
Signed-off-by: NAlex Converse <alex.converse@gmail.com>

Signed-off-by: NAlex Converse <alex.converse@gmail.com>

Signed-off-by: NAlex Converse <alex.converse@gmail.com>

aacenc: cosmetics, replace 'rd' with 'bits' in codebook_trellis_rate to make it more clear what is being calculated.
Signed-off-by: NAlex Converse <alex.converse@gmail.com>

Also fix a typo.
Signed-off-by: NAlex Converse <alex.converse@gmail.com>

Signed-off-by: NAlex Converse <alex.converse@gmail.com>

Macroify sanity checks and check return values of allocs and other functions.
Signed-off-by: NAlex Converse <alex.converse@gmail.com>

Signed-off-by: NKostya Shishkov <kostya.shishkov@gmail.com>

This matches the spec as well as the reference decoder, and fixes a bug
with interlaced frame decoding.
Signed-off-by: NKostya Shishkov <kostya.shishkov@gmail.com>

They have no code in common.

Reported and reviewed by Derek Buitenhuis.
Signed-off-by: NJanne Grunau <janne-libav@jannau.net>

The most recent demuxing/seeking code does not need them.

This uses the old demuxing code for OP1a and separate demuxing code for OPAtom.
Timestamp output is added to the old demuxing code.

The seeking code is made to seek to the start of the desired EditUnit only,
from which the normal demuxing code takes over (if OP1a). This means we
do not use delta entries or slices, only StreamOffsets. OPAtom seeking
basically works like before.

This also makes D-10 seeking behave the same way as OP1a and OPAtom. In other
words, we allow seeking before the start or past the end for D-10 too.

Based on several patches by Tomas Härdin <tomas.hardin@codemill.se> and
Reimar Döffinger <Reimar.Doeffinger@gmx.de>.

Changed av_calloc to av_mallocz, added overflow checks.

Signed-off-by: NJanne Grunau <janne-libav@jannau.net>

It is a really bad idea to assign a video codec id
when we have set codec_type to audio and vice versa.

Prevents detection of mp2 in mxf as mpeg2video.
Signed-off-by: NJanne Grunau <janne-libav@jannau.net>

This replaces the old essence_offset code.
Signed-off-by: NJanne Grunau <janne-libav@jannau.net>

Signed-off-by: NJanne Grunau <janne-libav@jannau.net>

This is not an error and expected behavior for OPAtom files.
Signed-off-by: NJanne Grunau <janne-libav@jannau.net>

This also zeroes new entries for good measure (used by future patches).
Signed-off-by: NJanne Grunau <janne-libav@jannau.net>

Signed-off-by: NJanne Grunau <janne-libav@jannau.net>

Specifically, this means parsing as before until we run into essence.
At that point we seek to the footer and parse until EOF. After that we start
seeking backward to the previous partition and parse that until we run into
essence or the next partition. This procedure is repeated until we encounter
the last partition we parsed in the forward direction.

The end result of all this is that large essence containers are not needlessly
parsed. This speeds up parsing large files a lot.
Signed-off-by: NJanne Grunau <janne-libav@jannau.net>

This fixes 0001GL.MXF.V1.mxf_opatom.mxf and 0001GL00.MXF.A1.mxf_opatom.mxf
getting two streams each due to both using the same SourcePackageID.
Signed-off-by: NJanne Grunau <janne-libav@jannau.net>

Signed-off-by: NJanne Grunau <janne-libav@jannau.net>

Signed-off-by: NJanne Grunau <janne-libav@jannau.net>

Based on patch from Tomas Härdin <tomas.hardin@codemill.se>
and work by Georg Lippitsch <georg.lippitsch@gmx.at>

Changed av_calloc to av_mallocz and added overflow checks.

Signed-off-by: NJanne Grunau <janne-libav@jannau.net>

Avoid modifying state if avio_seek fails.
Signed-off-by: NJanne Grunau <janne-libav@jannau.net>

This fixes reading of partition packs. The code stops reading after the
operational pattern and should skip the array of essence container
labels that follow.
Signed-off-by: NJanne Grunau <janne-libav@jannau.net>

Now that 0.8 is out we can reapply this commit. It breaks shared
avserver builds due to avserver using internal libavformat symbols,
which are now hidden, so this commit also disables avserver with
--enable-shared.

This avoids (for all practical cases) the issue of reusing
the same UDP port as for an earlier connection. If the remote
doesn't know the previous session was closed, he might keep
on sending packets to that port. If we always start off trying
to open the same UDP port, we might get those packets intermixed
with the new ones.

This is occasionally an issue when testing RTSP stuff with
DSS, perhaps also with other servers.
Signed-off-by: NMartin Storsjö <martin@martin.st>

Signed-off-by: NMartin Storsjö <martin@martin.st>

This check isn't relevant in the way the code currently works.

Also change a case of if (x == 0) into if (!x).
Signed-off-by: NMartin Storsjö <martin@martin.st>

The s->ssrc field is the sender's SSRC, we use ssrc + 1 to get
a collision free "unique" SSRC for ourselves in the RR part.
The SDES block in the RTCP packet should describe ourselves,
not the sender.

This was fixed for the RR part in 952139a3, but wasn't
fixed for the SDES part until now.

This could cause some Axis cameras to send RTCP BYE packets
to us due to the SSRC collision.
Signed-off-by: NMartin Storsjö <martin@martin.st>