Fix Coverity-Build  (#1499)

* Add omx dependency for Coverity-Build

* Add libva-dev dependency for Coverity-Build

* Add nvidia-cuda-dev dependency for Coverity-Build

* Enable tvhcsa and memoryinfo for Coverity-Build

* Disable mmal due to unavailability for Coverity-Build

Revert "Update debian/compat to version 10"

This reverts commit 2a370dd17fcac7e587d45fd9971e346536379ea3.

Ignore title mismatch if dup checking by CRID

Some channels add "New: " to the title if this is the first showing, so a title match with repeats will fail.

Update debian/compat to version 10

Compat version 10 was declared recommended in 2016.

An important change introduced in 10 was change of default to target
parallel builds

While version 13 is the currently recommended one, it is too new for
systems in use

https://github.com/Debian/debhelper/blob/5d1bb29841043d8e47ebbdd043e6cd086cad508e/debhelper.pod#compatibility-levels

Unify command time range to 10-300 ms

Unify the allowed range of command time
for rotors, switches and unicable devices.

Unify power up time range to 10-500 ms

Unify the allowed range of power up time
for rotors, switches and unicable devices.

Add configurable delays after Unicable operations

Allow user to adjust the length of time delays
after the Unicable device is powered up
and after a DiSEqC command is sent to it

Unify names and order of Unicable-specific configuration fields

Add descriptions to the existing Unicable configuration fields

update vaapi

- updated function _video_filters_get_filters() due to a bug hidden by previous bug (Bug #6247)
- fixed Decoder HW + Encoder HW --> was generating download / upload that increase CPU usage
- remove vp9 profile0 to 3 (ffmpeg vaapi is not supporting profiles for vp9)
- improve 'tier' and added: level, qmin, qmax, desired_b_depth
- printing Bitrate, Buffer size, Max bitrate (for each transcoding start)
- added superframe for vp9

descrambler: cccam: move send keepalive message to traces

Signed-off-by: Jaroslav Kysela <perex@perex.cz>

descrambler: cccam - simplify cccam_handle_keys()

Signed-off-by: Jaroslav Kysela <perex@perex.cz>

descrambler: cclient: optimization for multiple key clients

Do not use client when other handles requests.

Signed-off-by: Jaroslav Kysela <perex@perex.cz>

descrambler: cosmetic cleanups, more CAID logs

Signed-off-by: Jaroslav Kysela <perex@perex.cz>

descrambler: cwc: do not register bad provider numbers for betacrypt and irdeto

Signed-off-by: Jaroslav Kysela <perex@perex.cz>

descrambler: cwc: Fix the additional card registration (mgclient option in o*s*c*a*m)

It's expected that new CAIDs and providers are added to the current list rather
than to overwrite the previous fetched providers.

Signed-off-by: Jaroslav Kysela <perex@perex.cz>

grammar: Replace "then" with "than"

Preserve existing Unicable idnode during the set operation

Currently, the Unicable settings are always cleared
when enabling or disabling the corresponding satellite position
or its owning adapter. The reason is that the idnode with Unicable
configuration is always unconditionally deleted
during the "set" operation.

This change adds a check for the existing Unicable configuration
and preserves the existing idnode if its type remains the same.

The same logic is already used when changing the configuration
related to LNB, switch and rotor.

updated function _video_filters_get_filters()

- fixed Decoder HW + Encoder SW --> not working (with VAAPI) - Bug #6247
- fixed resize and deinterlace (*_vaapi for hw and ffmpeg for sw)

profile video resize improvements

- provide the ability to specify if scaling should be performed Up and Down, Up (only) or Down (only)

profile video resize improvements

- provide the ability to specify if scaling should be performed Up and Down, Up (only) or Down (only)

Add autorec duplicate handling default to dvr config.
Default will be applied to new autorecs unless overridden.
duplicate_handling.md: Add details of "Record if Unique" option.

update vaapi

- added new settings: platform, bitrate scale factor, low power, loop filter level, loop filter sharpness, async depth
- implemented new (dynamic) settings adjustment (in js)
- added new parameters: b, low_power, loop_filter_level, loop_filter_sharpness, async_depth
- tvhva_context_check_profile() will change TVHVAContext->entrypoint from VAEntrypointEncSlice into VAEntrypointEncSliceLP if VAEntrypointEncSlice is not available for that CODEC (according to VAAPI info)
- moved low_power to tvh_codeo_profile in order to initialize properly the entrypoint
- many error reporting improvements
- separated some bundled conditions

remove libavresample from build scripts

Add amd64 jammy to builds

Fix a few more builds, add kinetic support

Allow old builds to pass

Build for kinetic instead of impish

Remove variable declaration from for-loop

Don't fail on strict aliasing violations

Don't confuse GCC with zero-length array

Fix FTBFS introduced by 7b95ba4cf9113ae8808b3e4a9425010b607dbaca

Link: https://tvheadend.org/issues/6173
Link: https://tvheadend.org/issues/6226
References: 7b95ba4cf9113ae8808b3e4a9425010b607dbaca
Signed-off-by: Cédric Schieli <cschieli@gmail.com>

config: Enable HbbTV parser by default

It make sense to include it always when available

Link: https://tvheadend.org/issues/6223
Signed-off-by: Kacper Michajłow <kasper93@gmail.com>

dvb_psi_pmt: Recognize AC-4 audio descriptor

This allows pass-through of AC-4 audio stream.

Will need adjustment when AC-4 gets support in FFmpeg to enable
transcode.

Link: https://tvheadend.org/issues/6222
Signed-off-by: Kacper Michajłow <kasper93@gmail.com>

Add South Africa to Countries list.

dtv-scan-tables commit 28414c7 added muxes for South Africa.
Add the country to the country codes list.

Build various targets and prepare new repository  (#1476)

Some builds still need a newer python version

Avoid breaking strict aliasing in IP_AS_V{4,6}

GCC complains (one example, more in tcp.h):

In file included from src/main.c:41:
src/tcp.h: In function ‘ip_check_equal_v4’:
src/tcp.h:29:31: error: dereferencing type-punned pointer will break strict-aliasing rules [-Werror=strict-aliasing]
   29 | #define IP_AS_V4(storage, f) ((struct sockaddr_in *)&(storage))->sin_##f
      |                              ~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/tcp.h:67:14: note: in expansion of macro ‘IP_AS_V4’
   67 |     { return IP_AS_V4(a, addr).s_addr == IP_AS_V4(b, addr).s_addr; }
      |              ^~~~~~~~

storage (a) here is a pointer to sockaddr_storage which is either backed
by a sockaddr_in or sockaddr_in6 struct (here, it would be good if it is
the former, we decided based on sa_family in ip_check_equal). Referencing
it means we have a pointer to a pointer to sockaddr_storage here, which
we then cast to a pointer to sockaddr_in. Our so type-punned pointer is
then dereferenced breaking strict-aliasing as this pointer as well as
our original pointer share a memory location and both could be used to
change it even through they are of different types.

We can avoid this situation simply by removing the reference as then it
is just casting a pointer to a different type (which in this case is
legal as storage is really a sockaddr_in).

Removing the reference breaks users of the macro who do not feed it a
pointer to a sockaddr_storage, so while the warnings were all produced
by tcp.h, we end up changing code everywhere else to resolve them –
usually by just taking a reference or not dereferencing there.

As this is the only instance of (detected) strict-aliasing breakage
the Makefile is also changed to no longer build the entirety of
tvheadend with strict-aliasing rules disabled. Similar for the disabled
stringop warnings which weren't (correctly) triggered in the current
code.

As a small bonus, this also prevents gcc-11 from tripping with a false
positive over the previous tcp change in c0f616e / #1473.

Remove always-true checks

Reduce ADTS header size for better compatibility

Simplify IPv6 compare functions to unconfuse gcc compiler

In file included from src/tcp.c:32:
In function ‘ip_check_is_any_v6’,
    inlined from ‘ip_check_is_any’ at src/tcp.h:110:46,
    inlined from ‘ip_check_is_local_address’ at src/tcp.c:89:17:
src/tcp.h:105:57: warning: array subscript 1 is outside array bounds of ‘const struct sockaddr_storage[0]’ [-Warray-bounds]
  105 |   { return ((uint64_t *)IP_AS_V6(address, addr).s6_addr)[0] == ((uint64_t *)(&in6addr_any.s6_addr))[0] &&
      |            ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~
src/tcp.h: In function ‘ip_check_is_local_address’:
src/tcp.h:108:19: note: at offset 8 into object ‘address’ of size 8
  108 | static inline int ip_check_is_any(const struct sockaddr_storage *address)
      |                   ^~~~~~~~~~~~~~~

And more for the other half of the function and its three-times more or
less copy-paste instances. sockaddr_storage is not an array, but what
we actually do here is access s6_addr – which is an array of uint8_t.
Accessing the 16 uint8_t as 2 uint64_t apparently works, but not doing
it results in hopefully simpler to understand code for both humans and
compilers alike.

Don't call epg_broadcast_set_description twice

Don't crash the wizard if tvh has no inputs

If you have e.g. all but dvb disabled and forgot to plug in your usb
adapter the wizard would crash upon you as no inputs are defined and the
looping assumes there is at least one. Ideally a message should be shown
to indicate that you need inputs, but a wizard displaying a mostly empty
page is far better than a crash to discover your mistake.

Serve static html files with mimetype text/html

Use application/json instead of text/x-json as mimetype

The web has mostly agreed on this standardized mimetype and e.g.
trying to open an API URI with Firefox is now opened in its friendly
JSON viewer rather than downloaded.

epgdb: Resolve symlinks before using file location

The new epgdb is written to a temporary file and later renamed to
override the old epgdb file atomically. If you diverted the epgdb
to a different place away from your usual configuration (e.g. for
space and/or disk usage reasons like on an OpenWrt router) this
leads to overriding the symlink with a real file defeating the point.

By applying realpath on the path first we can resolve any symlink
along the path, while not considering it a failure if the epgdb
file doesn't exist yet. If on the other hand the path up to the file
doesn't exist we default to the old way of just taking the path
verbatim and let them be created by hts_settings_makedirs as before.

Note that this relies on the paths being sized PATH_MAX, which as the
manpage notes is POSIX.1-2001 conform, but broken by design as PATH_MAX
can't be relied upon, but the entire codebase makes heavy use of PATH_MAX
and there is a pre-existing usage of realpath() in this way so lets
pretend its okay for now.

References: b23686a55323625b15d4f99fd7af55259fa21828

Increase maximum ADTS packet size to match FFMPEG

iptv_auto: Add support for m3u "channel-number" tag

SiliconDust recently added m3u playlist generation support to the
HDHomeRun products.  They use "channel-number" for the channel
number, so add support for that value to TVHeadend.  It is used as
a fallback only if the previously-supported "tvh-chnum" and
"tvg-chno" tags aren't present.  This should be particularly useful
for usage with ATSC 3.0 channels, as those aren't supported in the
libhdhomerun library that TVHeadend normally uses for HDHomeRun
devices.  Without this, the channel number for all channels
imported from the HDHomeRun m3u is "0".

Fix race condition/data corruption in imagecache

Fix bad mono2sec usage

Attempt to fix profile sharer memory leak

May or may not works, let's see....

Attempt to fix HBBTV memory leak

Fix typo

Revert 4355488b8e1e868cb434bf95676c0944b44e88b3

Better fix was implemented in previous commit

Close FDs even if no UDP connection is used

Fixed typo

Update regexps for the finnish EIT scraping

mpegts dvb: Add support for LCN for provider DigiTV

output: UDP streaming

Fix potential memory leak

Ignore PCRE2 illegal accesses

See https://lists.exim.org/lurker/message/20160113.163710.006b34b9.en.html

Avoid leaking iptv fd's

Added support for ATSC text mode == 0x3F

Added support for ATSC text mode == 0x3F

handling of ATSC text mode == 0x3F (Select Unicode, UTF-16 Form) is added by calling atsc_utf16_to_utf8

+ change to bytecount instead of  srclen

Fix FTBFS introduced by 86f3617c8972c5362e51cee7d34cc2d69d799126

Fix crash when mpegts_service_refresh tries to open the CAT again

Added more 'text modes' to the ATSC Multiple String Structure decoder and convert text to UTF-8. (Fixes #5162)

 - Added support for decoding ATSC's "Multiple String Structure" text modes 0x1-0x6, 0x9-0x10, 0x20-0x27, 0x30-0x33.
 - Convert decoded text to UTF-8 instead of ISO-8859-1.
 - For unsupported 'compression types' or 'text modes' return a text string "[comptype=0x??,mode=0x??]" indicating
   the attempted compression type and text mode instead of the text segment.

Text output from ATSC's "Multiple String Structure" decoder should properly render in web browsers, specifically Unicode characters >= 0x80.

Allow network scan to modify muxes

When 'change muxes' option for network discovery is enabled, allow network scan to modify muxes rather than duplicate them on minor changes such as FEC

Fixed and cleanup the "PSIP: ATSC Grabber" module (Fixes #5610)

 - Bumped up limit on number of supported EIT/ETT tables from 5 to 256 (this is the max number of EIT/ETT tables in the ATSC specification)
 - Remove table type 4 as a valid ETT table, there is no support for this table here and table type 4 doesn't supply any EPG data--just long names for channels/services, etc.
 - Removed the very rapid toggling between mpegts_table_{add,destroy}(...) calls on EIT/ETT tables that caused continuous enabling/disabling of EIT and ETT PIDs.
 - Added an 'ETM Location' check to EIT table handling to ensure that an event's extra text is cleared in the EPG database if there is no matching ETT.
 - Removed epg_broadcast_change_finish(...) because it was erasing the extra text field in EPG database when updating title in EPG database (in EIT handler).
 - Added the ability for receiving ETT tables with the same version ID.  ETT tables use the same version ID for long runs of event IDs and 'Extra text's.
 - Fix bug in retrieving the number of tables EIT/ETT tables listed in the MGT table.

 - Removed the psip_{find,remove,add}_desc(...) functions (and supporting code) that stored ETT extra texts before
   there was a matching EIT event and then would try to match up the ETT extra texts when the EIT event showed up.
   However, the psip_*_desc(..) functions didn't keep track of the channel and would match up the ETT 'Extra text' with the wrong EIT event.

 - Removed some non-functional code.
 - Removed old commented out test code.

 - Added placeholder support for 'stop' and 'done' functions of this module for future development.

The "Over-the-air: PSIP: ATSC Grabber" module now quickly and correctly populates 'Title', 'Extra text', 'Start time', 'End time', 'Duration', etc... fields in the EPG Database.

Regexps for the finnish EIT scraping

 Changes to be committed:
new file:   fi

extending the regexps for the italian EIT scraping

Fix use-after-free

Building Tvheadend on Raspberry Pi with gcc 12 fails with this error

src/misc/json.c: In function ‘json_parse_string’:
src/misc/json.c:120:31: error: pointer ‘r’ used after ‘free’ [-Werror=use-after-free]
  120 |                   *failp = (a - r) + start;
      |                            ~~~^~~~
src/misc/json.c:118:19: note: call to ‘free’ here
  118 |                   free(r);
      |                   ^~~~~~~

This PR appears correct and fixes the gcc error but has not been tested as it is an error path.

No longer use git-protocol

add Access-Control-Allow-Headers content-length

Update Copyright year

Travis CI is dead, use GitHub actions for badge

GitHub actions improvements

Use GitHubs CI for Building

Add HMF_UUID to htsmsg_binary_write

Prevent deadlock-detector leaking memory

Remove useless null-check on an array

Fix potential null-pointer-dereference

Fix FTBFS in utils.c

U+0020 SPACE and U+00A0 NO-BREAK SPACE look the same, but they
aren't the same.

fix build with libressl

Fix the following build failure with libressl raised since
https://github.com/tvheadend/tvheadend/commit/e61acb8ad4a3411f4e7acfd8133d222299f6d47e:

utils.c:(.text+0x1614): undefined reference to `EVP_sha512_256'

Fixes:
 - http://autobuild.buildroot.org/results/cb18f6533806f3729f9718bdcc719384be375b66

Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>

Always parse 'src' in RTSP-requests

Originally, the code did not parse 'src' in RTSP-requests for
DVB-C tuners. This patch moves to always parse the parameter,
allowing e.g. Plex and possibly other SAT>IP-clients to work.

Signed-off-by: Nita Vesa <werecatf@outlook.com>

dvr_disk_space_cleanup() - do not return error if called again too soon (#1)

* Rework disk space check and cleanup

* Update dvr.h

* Update dvr_vfsmgr.c

Update for VAAPI transcoding

VAAPI: Buffer factor (buffersize manipulation) now configurable at WebUI.
VAAPI: Added rc_mode (h264/hevc) and tier (hevc), both configurable via WebUI.
VAAPI: Removed B-Frame workaround from the past at HEVC.
VAAPI: Added an option for ignore b-frames because some drivers needs this (AMD).
VAAPI: Also updated VP8 and VP9 encoding.
VAAPI: VP8 and VP9 now also reacts on options for "Buffer factor" and "Ignore B-Frames" and got the "force_key_frames" expression to increase picture quality.
VAAPI: Added destroy option for h264 codec (might have created memory issues)

SAT>IP client: UPnP header field names are case insensitive

- SAT>IP Protocol Specification 1.2.2: 3.3.2 Server Advertisements

More doozer build fixes

Fix doozer CentOS build

Fix sid doozer build

Update Python shebangs to python3

Update RPM to python3

Doozer fixes

Fix doozer builds

Fix some failing builds

Add --nowerror to build for RPM packages

Fix failing builds again

doozer: Migrate to Fedora 34 and 35

Update buffer size for h264 and hevc

Increased buffer for h264 and hevc transcoding (from ((self->bit_rate) * 1000) * 2 to ((self->bit_rate) * 1000) * 3).
With bigger buffer picture creates less artefacts on lower bitrates.

Changed debian package version to 7

Debian Testing does not support versions <7

epg: ignore past events when matching on eid

httpc: Fix multi-value "Connection" header checks

Connection header was checked for exact "close" or "upgrade" values
while it could contain multiple values delimited with commas.
New function was added for checking such cases.

Code is based on kv_find_value() function from:
https://cvsweb.openbsd.org/cgi-bin/cvsweb/src/usr.sbin/relayd/relayd.c

This fixes #6090

Episode number regexp

Proposed change to get also episode numbers given like "EP. nnn" (italian channels like Rai Gulp, Rai YoYo)