Some tables like EIT and ATSC EIT need to know what the value of table_id and
i_extension fields are for this PSI section. Therefor add this information to
the tables decoded structs.

The param struct was used after it was freed.

The p_prev->p_next was never set to NULL, but instead the p_pmt->p_next pointer was.
This resulted in never freeing additional PMT entries from the PAT.

When p->i_number == p_section->i_number AND p_prev == NULL, then the
first element of the list has to be replaced. Since it is the first
element we have to update the pointer to the list, which is pointing to
the first element
Signed-off-by: Jean-Paul Saman <jpsaman@videolan.org>

Decoding EIT event descriptors was off by 2 bytes in function dvbpsi_eit_sections_decode().

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

When using -Werror and -Wcast-align the use of sockaddr casting triggers a warning
about the cast saying: 'cast increases required alignment of target type' on ARM
and MIPS platform. A safer way to cast back and forth between sockaddr and sockaddr_in
(or sockaddr_in6) is using sockaddr_storage.

Repack DVBPSI_DECODER_COMMON to be byte aligned and add some checks to
do clever casting.
The TOT and SIS table structure definition need to be packed explicitly for use on ARM and MIPS platforms.

Remove empty remark.

In addition, my compiler complains about some non-sensical byte
overflow checks, which break my build (with -Werror enabled):
Signed-off-by: Jean-Paul Saman <jpsaman@videolan.org>

It seems that there was a bug introduced in renaming message
dvbpsi_message in dvbpsi_private.h. This is masked (in my environment
at least) by the configure script concluding that variadic macros are
not supported (i.e. ac_cv_cpp_variadic_macros=no) if -Werror is
included in the C flags, as the fragment in configure.ac used to
determine this generates a warning and thus failure.
Signed-off-by: Jean-Paul Saman <jpsaman@videolan.org>

Some distros (Ubuntu lucid) have slightly different glibc headers and require sys/types.h to be
included explicitly.

This struct was defined in two descriptors.

Found better solution.

This reverts commit 3b68bd17.

Make sure these definitions do not interact with each other.

When p->i_number == p_section->i_number AND p_prev == NULL, then the function
will crash with a segmentation fault. In this case the first element in the linked
list is going to be replaced. The pointer p_prev does not point to a valid sections,
since p is the first element in the list. To solve this case it needs to be treated
seperate.

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