libdvbv5: fix NIT structures

Add missing __attribute__((packed)) to nit structs.

As this is a binary description of the MPEG NIT table, it should not have
any padding on it.

Signed-off-by: André Roth <neolynx@gmail.com>
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

libdvbv5: VCT bitmap fixup

The size of bitfield1 is 32 bits. Fix it.

Signed-off-by: André Roth <neolynx@gmail.com>
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

libdvbv5: fix get_frontend logic

The patch that added a cache var for get/set broke the get frontend
data logic, as it doesn't copy data back from cache. Fix it.

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

dvb-scan: remove ISDB-T hack from it

This hack is not needed/desired, as it overrides what userspace
specified:
- if channel file is a DVBv3 file, those values are already
  filled by the compatibility standard code;
- if channel file is in DVBv5 format, it may override any
ISDB-T parameter.

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

dvb: simplify the logic that sets a (compat) DTV delivery system

Instead of using a loop, just call the existing function for
retrieve a parameter inside an entry.

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

dvb-fe: fix using a DVB-T file for ISDB-T

When a DVB-T channel file is used, and the frontend has ISDB-T
instead, put it into ISDB-T auto search mode.

That is better than let the Kernel to use the compatibility mode,
as the output files will have the proper ISDB-T fields.

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

libdvbv5: Only discard the services not found

If the PAT table describes a service ID, but it doesn't exist, just
discard it, and not all services after that.

This helps to be a little more resillient to broken PAT tables.

Caught with a test signal that was captured by a friend in Rio. The
tables describe a SD stream that it is not present on the TS.

$ dvbv5-scan ~mchehab/isdb-teste -F -G -T2
INFO     Scanning frequency #1 479142857
Lock   (0x1f) Signal= 78.83% C/N= 0.38% UCB= 252 postBER= 0
INFO     Service Globo HD, provider (null): digital television
INFO     Service Globo SD, provider (null): digital television
WARNING  Service ID 59201 not found on PMT!
INFO     Service Globo 1Seg, provider (null): user defined

Not sure if this is due to some capture error or not, but it is better
to be able to capture the 1Seg stream in any case, and just discard the
broken Service ID.

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

libdvbv5/dvb-fe: Fall back to v3 if needed

if FE_GET_PROPERTY gets error, fallback to DVBv3 stats

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

qv4l2: fix broken VBI support

Using qv4l2 with a vbi node broke when the pixel aspect ratio code was added.
Pixel aspect doesn't apply to vbi, and that wasn't checked.

Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>

libdvbv5: fix retrieve parameters for ISDB-T

ERROR    command MODULATION (4) not found during retrieve
ERROR    command INNER_FEC (9) not found during retrieve

Those parameters are per-layer, and not global.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

libdvbv5: Fix a warning at isdb_desc_partial_reception_free

descriptors.c:1175:3: warning: initialization from incompatible pointer type [enabled by default]
   .free  = isdb_desc_partial_reception_free,
   ^
descriptors.c:1175:3: warning: (near initialization for 'dvb_descriptors[251].free') [enabled by default]

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

v4l2-ctl: use libv4l2 wrapper for mmap as well.

The streaming code didn't use v4l2_mmap if the -w option was specified.
Correct this.

Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>

dvbv5-zap: allow using a frequency for a channel on exit after tuning

On exit after tuning, only the frontend is used. So, no need to specify
any PID. Just the transponder/channel frequency is enough.

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

dvbv5-zap: fix frontend status handling

Currently, check_frontend is doing nothing. Fix it to wait for a
lock, and remove the duplicated "frontend only" option, as this is
the same as exit after locking.

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

dvbv5-zap: on all pids mode, we can use the frequency

We don't need the service name on all PIDs mode.

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

dvb-fe: be a little less verbose with verbosity level 1

This is too much verbosity when scanning:

DEBUG    Stats for STATUS = 0
DEBUG    Stats for STATUS = 0
DEBUG    Stats for STATUS = 0
       (0x00) Quality= Poor Signal= -62.00dBm C/N= 19.00dB UCB= 497 postBER= 126x10^-18 PER= 86.9x10^-3
  Layer A: Quality= Good UCB= 0 postBER= 0 PER= 0
  Layer B: Quality= Poor UCB= 7030 postBER= 3.79x10^-3 PER= 71.9x10^-3

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

libdvbv5: Fix memory leak at TS info handler

parse_string allocates memory. Those need to be freed.

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

libdvbv5: fix memory leak at ISDB partial reception handler

This descriptor handles allocates memory but never frees it.

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

dvbv5-zap: fix all-pids record mode

the -P switch was not working fine, as the recorded mpeg file
were being corrupting. Fix it.

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

dvb/README: Remove some obsolete data

The descriptors handling is now completely different. Remove the
description of the old library from README file, as it doesn't apply
any more.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

libdvbv5: simplify VCT table parser

Do a cleanup at the code, simplifying it, and removing
the second pointer for the linked list.

That makes the code easier to analyse and review.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

libdvbv5: simplify SDT table parser

Do a cleanup at the code, simplifying it, and removing
the second pointer for the linked list.

That makes the code easier to analyse and review.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

libdvbv5: simplify PMT table parser

Do a cleanup at the code, simplifying it, and removing
the second pointer for the linked list.

While here, add support for multisection PMT.

That makes the code easier to analyse and review.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

libdvbv5: cleanup PAT table parser

Do a cleanup at the code, fixing coding style and moving
the head initialization to the beginning of the file.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

libdvbv5: simplify NIT table parser

Do a cleanup at the code, simplifying it, and removing
the second pointer for the linked list.

That makes the code easier to analyse and review.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

libdvbv5: Simplify PAT handling

Coverity thinks that there's a resource leak by having a pgm
not being properly stored.

While I don't agree with Coverity, the code could be simplified,
and use just one pointer instead of two, in order to fill the
programs linked list.

That makes it easier to be analyzed by both automatic tools and
by manual inspection.

So, simplify it.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

libdvbv5: fix some out-of-bound acesses on hexdump()

As Coverity points, there are some risk of getting out of bound
on some places. Fix it, and fix the coding style.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

libdvbv5: fix other error conditions for out of memory

As Coverity points, there are other places where this error also occurs.

Fix them.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

libdvbv5: pack NIT table

As all tables, this one should be packed.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

libdvbv5/dvb-scan: all tables should have an specified size

If a table has a zero size, reject it.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

dvb5-zap: Better handle the MPEG TS header

The code is now a little more elegant and easier to read.

No functional changes.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

libdvbv5: Fix other error conditions for out of memory

As Coverity points, there are other places where this error also occurs.

Fix all of them.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

libdvbv5: fix error condition on ISDBT delivery descriptor

It should return if an error occur.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

libdvbv5: fix error condition on ATSC service location descriptor

It should return if an error occur.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

dvbv5-scan: fix detection of duplicated freqs at the input file

As reported by Coverity, the dvb_scan_handler is always null on
that point. Just removing its check is enough to fix the issue.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

libdvbv5: don't use realloc() inside pat.c

The usage of realloc() inside pat is wrong, as the newly reallocated
data could have a different address, and this is not returned back
to dvb-scan's dvb_read_section_with_id().

While we could do some other tricks to pass the new pointer back,
the entire logic is really not good. Also, the PAT handler uses a
different model than the other tables.

So, change it to use a linked list for the programs. This can, however,
break the API for other applications using it.

Fortunately, the dvb_pat_program_foreach() macro abstracts this
API difference, so, the breakage will not actually happen, if the
other apps use this macro (with is the case of tvdaemon).

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

libdvbv5/dvb-scan: Remove dead code and add a check after realloc()

As reported by Coverity scan, the code just before the .init
if check warrants that tbl is always filled. So, checking it again
makes no sense. However, it makes sense to check after calling
realloc().

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

libdvbv5: change log level for the two nit handler messages

Those msgs are just warn for someone debuging the code. Don't
show them as errors.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

libdvbv5: fix terrestrial delivery constellation table

constellation == 2 means QAM64, according with ETSI EN 300 468 V1.11.1.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

libdvbv5: fix dvb-scan handler for T2 descriptor

The handler should take into account that not all descriptors
have all fields filled.

Also, while we don't have any sample of this descriptor
with a frequency table, better to not change the properties
for each new transponder's frequency.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

libdvbv5: fix T2 delivery descriptor handler

The data struct were not properly filled. Fix it.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

libdvbv5: fix extension descriptor handler

There are several issues at the extension handler:

1) the size is wrong, as it is getting data from
   descriptors.c, instead of from the extensions;

2) there are memory leaks;

3) It doesn't print the extension descriptors.

Fix them.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

dvbv5-zap: only handle SIGALRM if need

If timeout is not specified, or a SIGTERM/SIGINT was not received,
there's no need for handling SIGALRM.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

dvbv5-scan: handle SIGTERM/SIGINT nicely

Instead of just dying whenever a signal arrives, finish what it was
done. This way, already scanned channels will be written at the file.

At the worse case, after receiving a SIGTERM/SIGINT, it will die after
5 seconds, or after receiving a second SIGTERM/SIGINT.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

dvbv5-zap: exit nicely on SIGTERM/SIGINT

If the program is killed or aborted with CTRL-C, dies nicely.

That warrants that any data being processed to be handled, that
opened files will be closed and that the memory gets freed.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

dvbv5-zap: handle timeout also while in monitor mode

The timeout parameter specifies the maximum time where the zap will
be looping. This works everywhere, except while in monitor mode.

Fix it.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

libdvbv5/dvb-demux: fix CodingStyle

No functional changes.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

libdvbv5: clear data before setting a dmx filter

This fixes this Valgrind error:

==25172== Syscall param ioctl(generic) points to uninitialised byte(s)
==25172==    at 0x38CCAECEB7: ioctl (syscall-template.S:81)
==25172==    by 0x4043A8: dvb_set_pesfilter (dvb-demux.c:78)
==25172==    by 0x402F9D: do_traffic_monitor (dvbv5-zap.c:546)
==25172==    by 0x403AEA: main (dvbv5-zap.c:745)

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

dvbv5-zap: make Valgrind happy

Don't let leaks at the end: close all files, free all allocated
data.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

dvbv5-zap: allow using transponder frequency while in monitor mode

When this tool is using i monitor mode, we don't need audio or video
PID's. So, a "channel" type of file, with the transponders frequency,
plus the transponders parameters are enough. Add support for it.

This way, it is possible to do:

dvbv5-zap -I channel -c dvbc-test 573000000 -m

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

libdvbv5/descriptors.c: fix two warnings

descriptors.c:134:4: warning: format '%ld' expects argument of type 'long int', but argument 4 has type 'int' [-Wformat=]
    dvb_logwarn("descriptor type 0x%02x is truncated: desc len %ld, section len %zd",
    ^
descriptors/desc_frequency_list.c: In function 'dvb_desc_frequency_list_init':
descriptors/desc_frequency_list.c:31:12: warning: initialization discards 'const' qualifier from pointer target type [enabled by default]
  char *p = buf;

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

libdvbv5: some error handling improvements on dvb-file

All messages used during scan use the dvb_foo macros, excecpt for
those two places. Replace them.

Ok, ideally, all msgs should use this macro, but changing the other
ones is currently out of the scope.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

libdvbv5: Fix the size of the LCN descriptor

Instead of using the main struct, it was using the channel number
struct. That's bad, especially since the size is not enough to store
the struct dvb_desc data on it, causing reading paste buffers.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

libdvbv5: Improve descriptor error logs

When a descriptor is too small or too big, print a better log
message. Also, those are non-fatal errors. So, tag them as
WARNING.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

libdvbv5: handle NIT descriptors when verbose is not enabled

When verbose is enabled, the logic was supposed to check NIT
transport entries even without a callback, in order to print errors,
if a descriptor is found but not handled.

Unfortunately, the logic was wrong, effectively requiring verbose to
be on, in order to parse such descriptors.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

libdvbv5: get rid of dvb-scan-table-handler.[ch]

those files were useful during the transition to use the
descriptors library. However, after all code changes, they
became too tiny, and dependent of dvb-scan.[ch]. So, merge
the code and header back into them.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

libdvbv5: properly handle ISDB-T partial reception

Tag the 1seg service IDs as partial reception.

That's IMHO the last missing bit from the descriptors library
porting logic.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

libdvbv5: better implement the NIT parser

The code that were implementing the NIT parser were becoming too
complex, and had some common duplicated code.

Re-implement it, creating a helper routine to parse any NIT data,
and re-using the same routine to allow updating both the current
transponder and the detected transponders on a cleaner and saner
way.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

libdvbv5: remove the remaining bits from the NIT parsers

The previous NIT parser used to fill the entries data based on
the information detected on each transponder/channel.

Well, if the frontend was able to lock on the transponder, that
means that the parameters there at the entry are already correct
for the current channel, and that there's no need to use the NIT
table.

For new transponders frequencies, those data should be applied,
but this is done already inside dvb_add_scaned_transponders().

So, remove this code. As this is the last part using the old
descriptors logic, remove the struct definitions as well.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

libdvbv5: add support for DVB-T2 at NIT parser

Add support on dvb-scan for DVB-T2 transponder parsing.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

libdvbv5: add support for DVB-S/S2 at NIT parser

Add support on dvb-scan for DVB-S/S2 transponder parsing.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

libdvbv5: Re-add parser for ISDB-T NIT transponders addition

Add logic to handle NIT table for DVB-T on dvb-scan.c

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

libdvbv5: Re-add parser for ISDB-T NIT transponders addition

Add logic to handle NIT table for ISDB-T on dvb-scan.c

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

libdvbv5: Re-add NIT transponders addition

Move the logic that add new transponders into the library and add
support there for DVB-C.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

dvbv5-scan: move scanning code into dvb_scan_transponder()

This makes the scan library more complete, and allows reusing the
code there into other applications.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

libdvbv5/dvb-file: describe the service type on the logs

While we currently don't store the service type on the channel
file, it could be useful for the use to know what was found. So,
add the information there at the output information.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

libdvbv5: improve debug messages

Add several improvements at the debug messages:

- Without verbose, show only the Virtual channels or
  services to be stored in the file;

- instead of using fprintf for errors, use the error
  macros. That will make errors colored;

- With verbose=1, will show all parsed data;

- With verbose=2, will show all unparsed descriptors and
their hexdump contents;

- With verbose=3, will show all descriptors and their
hexdump contents, including the ones without a parser.

Verbose levels 2 and 3 helps to detect if some improvements would
be needed inside the descriptor parser's library.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

libdvbv5: shut up a few compilation warnings

remove a couple compilation warnings:

descriptors/desc_extension.c:148:3: warning: passing argument 3 of 'init' from incompatible pointer type [enabled by default]
   init(parms, p, desc, ext->descriptor);
   ^
descriptors/desc_extension.c:148:3: note: expected 'struct dvb_extension_descriptor *' but argument is of type 'struct dvb_desc *'
descriptors/desc_extension.c: In function 'extension_descriptor_free':
descriptors/desc_extension.c:154:19: warning: unused variable 'desc' [-Wunused-variable]

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

libdvbv5: be sure to not go past buffer on VCT parsing

Add checks to avoid read more data than available on buffer.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

libdvbv5: be sure to not go past buffer on SDT parsing

Add checks to avoid read more data than available on buffer.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

libdvbv5: be sure to not go past buffer on PMT parsing

Add checks to avoid read more data than available on buffer.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

libdvbv5: Add missing dvb_dmx_stop() on error handling

If the table can't be loaded, stop streaming.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

libdvbv5: don't cause crash if NIT is missing on fe-file.c

If NIT table is not read, it is not possible to get the vchannel
data from it.

Instead of crashing, fails nicely.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

libdvbv5: fix debug msg for Network PID

When program_number is 0, the PID refers to the Network PID,
according with ISO 13.818-1. Fix it.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

libdvbv5: add multisection support for PAT table

As PAT can be multisection, add support for it.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

libdvbv5: be sure to not go past buffer on PAT parsing

Add checks to avoid read more data than available on buffer.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

libdvbv5: avoid nit to read out of the buffer

As reported by Valgrind:
==6709== Invalid write of size 8
==6709==    at 0x4099F2: dvb_parse_descriptors (descriptors.c:61)
==6709==    by 0x40B9CB: dvb_table_nit_init (nit.c:92)
==6709==    by 0x408F07: dvb_read_section_with_id (dvb-scan.c:200)
==6709==    by 0x40946A: dvb_get_ts_tables (dvb-scan.c:80)
==6709==    by 0x4025E2: run_scan (dvbv5-scan.c:477)

Sometimes, we're reading past of the buffer on NIT parser.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

libdvbv5/dvb-file: Don't leave memory leaks on channel store failure

If channel store fails, the channel var should be freed.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

libdvbv5/descriptors: don't leak memory on error

Check buffer size before alloc, to avoid memory leaks.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

libdvbv5/descriptors: Fix extension descriptor handling

The extension descriptor handling was wrong. Rewrite the code,
in order to make it right.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

libdvbv5/descriptors: Fix error handling routine

the "current" structure is not allocated yet. Don't use it too
early.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

libdvbv5/dvb-file: Fix the logic for program found check

The logic is broken, causing a crash. Fix it.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

libdvbv5/dvb-scan: require PAT also for ATSC

If the PAT table is not found, it can't proceed, even for ATSC.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

dvb/dvbv5-zap: Fix help message

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

libdvbv5/dvb-scan: avoid memory leak on table read timeout

Free buf, if table read fails with timeout.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

libdvbv5/dvb-file: Allow using SDT for ATSC

When SDT table parsing is forced, allow using it for ATSC.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

dvbv5-scan: release resources on errors

If an error occurs, release memory and close the file descriptor,
if opened.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

dvb-fe: don't cause memory leak on open errors

If open fails, release memory.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

libdvbv5: don't leak memory on ATSC service location

Call free routine for desc_atsc_service_location, in order
to avoid memory leaks.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

libdvbv5: On ATSC, only try to get SDT if VCT is not found

As, on ATSC, channels are described on VCT, don't try to read
SDT by default, if VCT is found.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

libdvbv5: add support for ATSC VCT table parsing

On ATSC, the VCT table is used to describe the channels,
and SDT is generally not implemented.

Implement a function to fill the channel data from it, if
available. Otherwise, fall back to SDT parsing.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

Fix desc_frequency_list descriptor

This descriptor is commented. Fix it.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

libdvbv5/dvb-file: fix parameters for dvb_vchannel()

Passed parameters were wrong. Fix them.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

libdvbv5/dvb-file: fix allocation of dvb_file

Correct a few allocation issues reported by Valgrind:

- Size of dvb_file is wrong;
- getline requires a free after end using it;
- entries should be freed at dvb_file_free().

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

libdvbv5/dvb-file: use the new descriptors for virtual channel

Search for the LCN and TS info descriptors at the NIT table, for
the virtual channel information.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

libdvbv5: produce dvb data from the new structures

Now that dvb-scan uses the new descriptor structures, convert
dvb-file to rely on them to produce the scanned file data.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

libdvbv5/dvb-scan-table-handler: remove free of unallocated structs

While those old structs are still part of struct dvb_v5_descriptors,
they are not allocated anymore. So, we don't need to fix them. Get
rid of those structs.

We can't remove them from the main struct yet, as dvb-file and
dvbv5-scan still use.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

libdvbv5/dvb-scan: fix memory leak

As pointed by Valgrind, there are still some memory leaks at
dvb_get_ts_tables().

That happens because pmt is allocated by dvb_read_section().
Fix it.

While here, fix the error condition to not mangle with the main
dvb_scan_handler->program pointer.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

libdvbv5/descriptors: descriptors data should always be freed

It is up to each descriptor's free function to free data allocated
during its init phase. However, the container of the descriptor
itself was allocated by dvb_parse_descriptors(). So, it should be
up to dvb_free_descriptors() to deallocate.

This error was detected by Valgrind.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

libdvbv5/dvb-fe: make valgrind happy

clean prop before using it. This makes valgrind happy.

Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>