Opened 13 years ago

Closed 13 years ago

#2874 closed enhancement (fixed)

add monitoring of decryption to DTVSignalMonitor

Reported by: Janne <janne-mythtv@…> Owned by: danielk
Priority: minor Milestone: 0.21
Component: mythtv Version: head
Severity: medium Keywords:
Cc: Ticket locked: no

Description

Tested with encrypted DVB Streams and DVBRecorder/DVBCam

but it should also work for other DTV Streams which are misleadingly marked as encrypted in the PMT.

Attachments (5)

encryption_sigmon.diff (18.2 KB) - added by Janne <janne-mythtv@…> 13 years ago.
2874-v1.patch (17.1 KB) - added by danielk 13 years ago.
Minor formatting + compilation fix for HDHomeRun
encryption_sigmon_v2.diff (24.1 KB) - added by Janne <janne-mythtv@…> 13 years ago.
2874-v4.patch (28.1 KB) - added by danielk 13 years ago.
Enhanced version of patch which tracks encryption for each program in PAT.
mythtv-setpmt.patch (597 bytes) - added by yeasah@… 13 years ago.
SetPMT called too often on some networks

Download all attachments as: .zip

Change History (14)

Changed 13 years ago by Janne <janne-mythtv@…>

Attachment: encryption_sigmon.diff added

comment:1 Changed 13 years ago by danielk

Owner: changed from Isaac Richards to danielk

Changed 13 years ago by danielk

Attachment: 2874-v1.patch added

Minor formatting + compilation fix for HDHomeRun

Changed 13 years ago by Janne <janne-mythtv@…>

Attachment: encryption_sigmon_v2.diff added

comment:2 Changed 13 years ago by Janne <janne-mythtv@…>

Updated patch attached:

  • checks status of all video and audio streams
  • a stream is unencrypted if 8 successive unencrypted packets are seen
  • moved most of the logic to mpegstreamdata
  • fixed compilation for disabled dvb

The active eit scan checks also the encryption status. I didn't wanted to avoid this by a hack but change the tuning for the eit scan. The active eit scan doesn't need a PMT to work. I'll change the tuning for the active eit scan

Daniel, I hope I got all changes from your patch.
I saw it only when I wanted to attach my patch. It's a pity that trac doesn't send an email after a new file is attached (at least to subscribed persons).

comment:3 Changed 13 years ago by danielk

Milestone: unknown0.21

This appears to work well here, I'm going to wait a few days before committing just because these latest changes haven't been in your tree very long. I'll start working on #2830 which depends on this today though.

Changed 13 years ago by danielk

Attachment: 2874-v4.patch added

Enhanced version of patch which tracks encryption for each program in PAT.

comment:4 Changed 13 years ago by danielk

Janne, I've attached an updated version of this patch.

It tests the decryption status of each program, whether or not we are sending any data to the CAM for it. This should fix allow us to view channels that are reported to be encrypted in the PMT but not in the actual streams.

This also sends the program number in the HandleEncryptionStatus?() callback.

comment:5 Changed 13 years ago by Neeesat

I would be very pleased if someone could make possible to use the patch for .20 with fixes.

Regards

Neeesat

comment:6 Changed 13 years ago by danielk

Resolution: fixed
Status: newclosed

(In [12473]) Fixes #2874. Adds monitoring of TS packet encryption status.

This should avoid getting stuck on a channel when using a CAM, and allow tuning to channels that claim to be encrypted when they are not without unchecking the "Ignore Encrypted Channels" checkbox.

This does not address encryption when scanning for channels but is a dependency for that functionality.

comment:7 Changed 13 years ago by Janne Grunau

(In [12474]) convert encryption status to string in VERBOSE macro. Refs #2874

Changed 13 years ago by yeasah@…

Attachment: mythtv-setpmt.patch added

SetPMT called too often on some networks

comment:8 Changed 13 years ago by yeasah@…

Resolution: fixed
Status: closedreopened

Some networks have multiple PMTs on the same PID -- the existing code will pass all the PMTs it sees right on to the CAM during the signal monitor phase. This patch just adds a check to make sure the PMT is for the right program to avoid sending undesired PMTs to the CAM.

comment:9 Changed 13 years ago by Janne Grunau

Resolution: fixed
Status: reopenedclosed

(In [12476]) Fixes #2874, Refs #2703, send only the PMT off the tuned program to the CAM

Note: See TracTickets for help on using tickets.