Modify

Ticket #5603 (closed patch: fixed)

Opened 4 years ago

Last modified 3 years ago

programid's and seriesid's should not be used without an authority

Reported by: David Waring <david.waring@…> Owned by: janne
Priority: minor Milestone: 0.22
Component: eit Version: head
Severity: low Keywords:
Cc: stuarta, stuartm Ticket locked: no

Description

According to ETSI TS 102 323, CRIDs in content_identifier_descriptor's must have an explicit authority or a default_authority found in the SDT, BAT or NIT.

MythTV uses the identifier anyway even though it may conflict with other services without a default_authority set and give false programme and series matches.

This patch corrects this by ignoring truncated CRIDs when no default authority can be found. The patch will strip the "crid://" part from fully specified CRIDs. The patch also adds the ability to obtain the default authority from the first or second loops in the NIT and stores these, if present, with the multiplex information.

(NIT authorities tested using simulated Crystal Palace signal)

Attachments

network_default_authority.patch (25.3 KB) - added by David Waring <david.waring@…> 4 years ago.
Patch for CRIDs without authorities and NIT default authorities

Change History

Changed 4 years ago by David Waring <david.waring@…>

Patch for CRIDs without authorities and NIT default authorities

comment:1 Changed 4 years ago by stuarta

  • Cc stuarta added

comment:2 Changed 4 years ago by stuartm

  • Cc stuartm added

comment:3 Changed 4 years ago by stuarta

In a related ticket #5562, we aren't populating the default authority descriptor when scanning astra (or probably any dvb-s for that matter)

comment:4 Changed 4 years ago by David Waring <david.waring@…>

The problem there is that there is no scan whatsoever of the BAT. The retrieval of the default authority from there should behave in a similar way to the NIT default authority in my patch. So I'd suggest this patch should be a requirement of any fix for #5562.

I'm trying to add BAT parsing into my local copy and may have a patch for #5562 soon. I think I have made the relevant changes but I'm just having a bit of trouble getting the Hauppauge HVR-4000 card to work properly, so I've been unable to test so far.

comment:5 Changed 4 years ago by stuarta

I've compiled the BAT processing on ticket #5530 as a starting point to getting this all working.

However i've yet to get as far as testing it.

Stuart

comment:6 Changed 3 years ago by stuartm

  • Owner changed from ijr to stuarta
  • Status changed from new to assigned
  • Component changed from mythtv to eit

comment:7 Changed 3 years ago by stuarta

(In [20048]) Refs #5603. DB Schema changes for default_authority changes.

  • Removes programid's and seriesid's that don't contain a default authority
  • Adjusts the size of programid & seriesid to cope with the maximum size they can be sent as
  • Adds default_authority column to both dtv_multiplex and channelscan_dtv_multiplex

comment:8 Changed 3 years ago by stuarta

(In [20049]) Refs #5603. EIT portion of the changes.

If we can't find a default_authority for the channel either on the channel itself or on the multiplex, then the programid / seriesid is invalid, so we don't store that information into the program data.

comment:9 Changed 3 years ago by stuarta

  • Status changed from assigned to started

comment:10 Changed 3 years ago by janne

  • Owner changed from stuarta to janne
  • Status changed from started to accepted

comment:11 Changed 3 years ago by janne

  • Status changed from accepted to closed
  • Resolution set to fixed

rest of this patch will be handled in #5562 especially since the patch is broken after the channelscan merge

View

Add a comment

Modify Ticket

Action
as closed
The resolution will be deleted. Next status will be 'new'
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.