Skip to content
This repository

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP
Browse code

Fixes #10054. Fixes handling of segmented SDT and PAT tables.

The code was attempting to verify if it had seen an SDT for
a given TSID before, and it inadvertantly created an empty
entry rather than just verifying it was missing.

This leads to further errors when the scanner thinks it has
an SDT for a transport when it is in fact missing it.

The same logic issue was found in the PAT handling.

Based on original patch by
Argus <pthorn-mythd at styx2002.no-ip.org>
  • Loading branch information...
commit 778d8111e8ffd61118cf36262eb0a1422a70ffd2 1 parent 19df7fb
Stuart Auchterlonie authored March 23, 2012
4  mythtv/libs/libmythtv/channelscan/channelscan_sm.cpp
@@ -752,7 +752,7 @@ bool ChannelScanSM::UpdateChannelInfo(bool wait_until_complete)
752 752
         if (tsid_checked[tsid])
753 753
             continue;
754 754
         tsid_checked[tsid] = true;
755  
-        if (!currentInfo->pats[tsid].empty())
  755
+        if (currentInfo->pats.contains(tsid))
756 756
             continue;
757 757
 
758 758
         if (!wait_until_complete || sd->HasCachedAllPAT(tsid))
@@ -805,7 +805,7 @@ bool ChannelScanSM::UpdateChannelInfo(bool wait_until_complete)
805 805
         if (tsid_checked[tsid])
806 806
             continue;
807 807
         tsid_checked[tsid] = true;
808  
-        if (!currentInfo->sdts[tsid].empty())
  808
+        if (currentInfo->sdts.contains(tsid))
809 809
             continue;
810 810
 
811 811
         if (!wait_until_complete || sd->HasCachedAllSDT(tsid))

0 notes on commit 778d811

Please sign in to comment.
Something went wrong with that request. Please try again.