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>

Using the current date for capture time makes no sense when the source is a file.

MPEG TS streams with multiple programs got squashed when printing the summary.
Since it did not handle the correct PMT it also failed to properly calculate PCR duration.

Handling the PMT explicitly allowed calculating the correct PCR duration and bitrate for all
streams in a PMT.

Rework checking section boundaries.

Check the needed bytes and available data bytes in dvbpsi_sis_cmd_splice_insert_decode().

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>

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>

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

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

The splice_insert command decoder is implemented in function dvbpsi_sis_cmd_splice_insert_decode().

Rename struct dvbpsi_sis_component_utc_splice_time_t to dvbpsi_sis_component_t and
also rename dvbpsi_sis_splice_event_t::p_splice_utc_time to dvbpsi_sis_splice_event_t::p_component.

The code meant to move the upmost pts_adjustment bit 33 down to the
lowest bit (bit 1), so it can be or-ed to a byte as lowest bit. However
the first part of the calculation (i_pts_adjustment & 0x00800) had as result
zero. Then shifting it over 32 bit, resulted in still zero. The calculation
was plain wrong.

First shift all bits over 32 to the right. This places bit 33 on bit 1, then
mask out all the other bits and keep bit 1, thus; (i_pts_adjustment >> 32) & 0x01

The function dvbpsi_DetachDemux(), which is now a wrapper around dvbpsi_chain_demux_delete()
failed to check the return value. Now it returns an error message when the return value is 'false'.

Dereference after NULL check (Fixes CID #134926, #134927, #134928, #13429, #13430, #134931, #134932, #134933, #134934, #134935, #134936, #134937, #134938, #134939, #134940)

Check the dvbpsi handle before passing it to dvbpsi_chain_demux_delete().

(cherry picked from commit cad97408)
Signed-off-by: Jean-Paul Saman <jpsaman@videolan.org>