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

The 3 first bytes of the secion are already skipped in dvbpsi.c
`p_section->p_payload_start = p_section->p_data + 3;`

p_section->p_payload_start therefore points to the protocol_version field of the splice_info_section

From SCTE 35:2019: The section_syntax_indicator is a 1-bit field that should always be set to ‘0’
From ISO 13818-1: if the indicator is set to '0', then only the fields 'table_id' through 'private_section_length' shall follow the common structure syntax and semantics and the rest of the private_section may take any form the user determines.

ANSI/SCTE 35 2019r1: The section_syntax_indicator is a 1-bit field that should always be set to ‘0’, indicating that MPEG short sections are to be used.

According to ISO 13818-1: The continuity_counter shall not be incremented when the adaptation_field_control of the packet equals '00' or '10'.

otherwise that's pretty useless in SI flow

fixes regressions by commit
7e9f4ed3
(early multi sections table end, segmented EIT failure)

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

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

The function dvbpsi_chain_demux_delete() failed to call p->pf_del() for all
attached decoder. This resulted in leaked decoder memory.

SCTE35 / CUEI Descriptor parsing bug. Descriptor lengths of one byte fail to parse, instead returning null.

The Spec from 2004 to 2016 shows the descriptor payload has never been
more than a single byte.
See SCTE 35 2016 Page 19 Table 2 (Section 8.2).

Correct the length validation check, and reject any descriptor that
IS NOT one byte long, as per the spec.
Signed-off-by: Jean-Paul Saman <jpsaman@videolan.org>

src/demux: functions dvbpsi_*DemuxSubDecoder() are no longer implemented, but are merely empty stubs

The function dvbpsi_decoder_psi_section_add() never set p_decoder::i_last_section_number when
adding a new section at the end of the list. This was left to the decoder implementations. However
the correct action would have been to set p_decoder::i_last_section_number in dvbpsi_decoder_psi_section_add().