catalinii / minisatip

minisatip is an SATIP server for linux using local DVB-S2, DVB-C, DVB-T or ATSC cards
https://minisatip.org
324 stars 78 forks source link

Old packets left in DDCI buffer sometimes #1076

Closed Jalle19 closed 1 year ago

Jalle19 commented 1 year ago

Lately I've noticed that sometimes when I switch to a channel, the switch happens very quickly and K get about a second of old video/audio from the last time the channel was active. Could be a regression since introducing the new FIFO buffer? @catalinii have you noticed something similar?

Jalle19 commented 1 year ago

It's of course also possible it's my tvheadend that is acting up.

catalinii commented 1 year ago

I think this is possible to happen:

I did not pay attention and it probably can happen only if the old and new channel shares the same pids

Jalle19 commented 1 year ago

I've only seen it when switching back to a channel that was previously tuned, i.e. you never get another channel's data, always stale data from the same channel.

Any way to fix this?

Jalle19 commented 1 year ago

Gonna try reverting the FIFO changes locally to see if it helps. It seems to be easy to reproduce - just switch between a CAM channel and a free channel.

Jalle19 commented 1 year ago

Reverting seems to fix the issue.

catalinii commented 1 year ago

Thx for the report

catalinii commented 1 year ago

Can you send me a quick repro?

Jalle19 commented 1 year ago

Like I said, just switch between a CAM channel and a free channel. When you switch back to the encrypted channel it will start at the same place it started the last time.

catalinii commented 1 year ago

can you try next branch ? It should be fixed

Jalle19 commented 1 year ago

Which commit is supposed to fix it? :thinking:

catalinii commented 1 year ago

Sorry forgot to push it.: https://github.com/catalinii/minisatip/commit/303ab856620e6676178efa538f4c7ad34be5dfb4

Jalle19 commented 1 year ago

Thanks, I'll try to find time to test next week

Yuri666 commented 1 year ago

For me it fixed

Jalle19 commented 1 year ago

Seems fixed indeed, thanks @catalinii !