[media] dvb_demux: Transport stream continuity check fix
authorJohn Smith <johns90812@gmail.com>
Tue, 5 Mar 2013 21:02:43 +0000 (18:02 -0300)
committerMauro Carvalho Chehab <mchehab@redhat.com>
Tue, 19 Mar 2013 18:14:06 +0000 (15:14 -0300)
This patch avoids incrementing continuity counter
demux->cnt_storage[pid] for TS packets without payload in accordance
with ISO /IEC 13818-1.

[mchehab@redhat.com: unmangle whitespacing and fix CodingStyle.
 Also checked ISO/IEC spec: patch is according with it]
Reviewed-by: Mauro Carvalho Chehab <mchehab@redhat.com>
Signed-off-by: John Smith <johns90812@gmail.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
drivers/media/dvb-core/dvb_demux.c

index d319717eb535fcefcdd983b18fd39f3d6d1c9c24..71641b2dde6b5a1f1073ac4272647153252e85c2 100644 (file)
@@ -440,20 +440,22 @@ static void dvb_dmx_swfilter_packet(struct dvb_demux *demux, const u8 *buf)
                if (!dvb_demux_feed_err_pkts)
                        return;
        } else /* if TEI bit is set, pid may be wrong- skip pkt counter */
-       if (demux->cnt_storage && dvb_demux_tscheck) {
-               /* check pkt counter */
-               if (pid < MAX_PID) {
-                       if ((buf[3] & 0xf) != demux->cnt_storage[pid])
-                               dprintk_tscheck("TS packet counter mismatch. "
-                                               "PID=0x%x expected 0x%x "
-                                               "got 0x%x\n",
+               if (demux->cnt_storage && dvb_demux_tscheck) {
+                       /* check pkt counter */
+                       if (pid < MAX_PID) {
+                               if (buf[3] & 0x10)
+                                       demux->cnt_storage[pid] =
+                                               (demux->cnt_storage[pid] + 1) & 0xf;
+
+                               if ((buf[3] & 0xf) != demux->cnt_storage[pid]) {
+                                       dprintk_tscheck("TS packet counter mismatch. PID=0x%x expected 0x%x got 0x%x\n",
                                                pid, demux->cnt_storage[pid],
                                                buf[3] & 0xf);
-
-                       demux->cnt_storage[pid] = ((buf[3] & 0xf) + 1)&0xf;
+                                       demux->cnt_storage[pid] = buf[3] & 0xf;
+                               }
+                       }
+                       /* end check */
                }
-               /* end check */
-       }
 
        list_for_each_entry(feed, &demux->feed_list, list_head) {
                if ((feed->pid != pid) && (feed->pid != 0x2000))