(cherry picked from commit fde118a1)

The EIT tables were output incomplete as the
first sections were always dropped.

(2 sections, last_section == 1)
Ex: section 1
    section 0
    secton  1

IsCompleteEIT is always true in first condition in that case.
It is also always true if section is N/N.
first_section > -1 && section_number == last_section because
it has just been added.

Doing the check before effectively adding the section
allows a correct evaluation against current section.
Signed-off-by: Jean-Paul Saman <jpsaman@videolan.org>

dvbpsi_GenVStreamDr set the MPEG_1_only_flag incorrectly : it should be
set only if the descriptor does not carry extra information about MPEG-2
video. The previous behaviour was exactly the opposite.
Signed-off-by: Jean-Paul Saman <jpsaman@videolan.org>

Remove dead debug code.

Fix dereference before NULL check.

Signed-off-by: Jean-Paul Saman <jpsaman@videolan.org>

Signed-off-by: Jean-Paul Saman <jpsaman@videolan.org>

This patch adds support for the MPEG-4 audio descriptor. aac_profile.h is now
allowed to be included multiple times, since it is also used by dr_1c.h.
Signed-off-by: Jean-Paul Saman <jpsaman@videolan.org>

Signed-off-by: Jean-Paul Saman <jpsaman@videolan.org>

Signed-off-by: Jean-Paul Saman <jpsaman@videolan.org>

The definition of AAC profile and level values is also used by the MPEG-4 audio
descriptor. Moving this definition to a separate file eliminates unnecessary
code duplication. New values from the 2015 version of H.222 were also added.
Signed-off-by: Jean-Paul Saman <jpsaman@videolan.org>

Just ignore as we can't handle it.
Signed-off-by: Jean-Paul Saman <jpsaman@videolan.org>

Private tables on PMT are common (ATSC) and should
not trigger an error.
Signed-off-by: Jean-Paul Saman <jpsaman@videolan.org>

Signed-off-by: Jean-Paul Saman <jpsaman@videolan.org>

Signed-off-by: Jean-Paul Saman <jpsaman@videolan.org>

Signed-off-by: Jean-Paul Saman <jpsaman@videolan.org>

dvbinfo now checks whether the libdvbpsi decode functions could properly decode
a descriptor before actually passing it to the dump function.
Signed-off-by: Jean-Paul Saman <jpsaman@videolan.org>

The address of a local variable instead of its actual value was passed
to memcpy. As a result, the actual memory referenced by the
p_additional_info pointer was never written to when decoding the
descriptor. Analogous issue occurred when generating.
Signed-off-by: Jean-Paul Saman <jpsaman@videolan.org>