Thread: [xineliboutput-devel] bug in ts2es.c
Brought to you by:
phintuka
From: zaverel <za...@fr...> - 2012-10-29 19:26:08
|
Hello, commit: demux-ts: flush complete PES packets immediately don't seem good http://xineliboutput.cvs.sourceforge.net/viewvc/xineliboutput/vdr-xineliboutput/xine/ts2es.c?r1=1.16&r2=1.17&sortby=date it break h264 video from iptv plugin. Only sound work. When i play xine http://127.0.0.1:37890 that work (video+audio) If i reverted the commit , video+audio work like before with vdr-sxfe. See you. |
From: Petri H. <phi...@us...> - 2012-11-14 12:02:24
|
On ma, 2012-10-29 at 20:26 +0100, zaverel wrote: > Hello, > > commit: demux-ts: flush complete PES packets immediately don't seem good > > http://xineliboutput.cvs.sourceforge.net/viewvc/xineliboutput/vdr-xineliboutput/xine/ts2es.c?r1=1.16&r2=1.17&sortby=date > > > it break h264 video from iptv plugin. > Only sound work. > > When i play xine http://127.0.0.1:37890 that work (video+audio) > > > If i reverted the commit , video+audio work like before with vdr-sxfe. Attached patch (fix1.diff) should fix this. If not, fix2.diff disables that functionality for video streams. If fix1 does not help, maybe you could provide short sample recording (something like one second should be OK) ? Or some log output with attached log.diff patch ? - Petri |
From: zaverel <za...@fr...> - 2012-11-14 15:25:16
Attachments:
fix1.log
|
Le 14/11/2012 13:02, Petri Hintukainen a écrit : > Attached patch (fix1.diff) should fix this. If not, fix2.diff disables > that functionality for video streams. > If fix1 does not help, maybe you could provide short sample recording > (something like one second should be OK) ? Or some log output with > attached log.diff patch ? > > > - Petri > Hello, fix1.diff don't work. fix2.diff work. log.diff seem to have a typo that don't compile with this->pes_length but is ok with this->pes_leng i attach fix1.log and a link to a sample recording http://dl.free.fr/jD6c3V0t6 see you |
From: zaverel <za...@fr...> - 2013-01-09 16:37:35
|
Le 07/01/2013 09:57, Petri Hintukainen a écrit : > On ke, 2012-11-14 at 16:25 +0100, zaverel wrote: >> Le 14/11/2012 13:02, Petri Hintukainen a écrit : >>> Attached patch (fix1.diff) should fix this. If not, fix2.diff disables >>> that functionality for video streams. >>> If fix1 does not help, maybe you could provide short sample recording >>> (something like one second should be OK) ? Or some log output with >>> attached log.diff patch ? >>> >>> >>> - Petri >>> >> Hello, >> >> fix1.diff don't work. >> >> fix2.diff work. >> >> log.diff seem to have a typo >> >> that don't compile with >> this->pes_length but is ok with this->pes_leng >> >> >> i attach fix1.log and a link to a sample recording >> >> http://dl.free.fr/jD6c3V0t6 >> > Thanks for the log and sample, it helped me to find the root cause of > this problem. > > This should be fixed in CVS now. > > > - Petri > > > hello, i'm afraid that's not perfect: Now video is good but i have random crash between 2 and 3 mn when i am on iptv channel and only iptv channel. this is the log just before the crash video_out: throwing away image with pts 12930689 because it's too old (diff : 3744). video_out: throwing away image with pts 12934352 because it's too old (diff : 9146). video_out: throwing away image with pts 12938012 because it's too old (diff : 14568). video_out: throwing away image with pts 12945682 because it's too old (diff : 25124). video_out: throwing away image with pts 12953558 because it's too old (diff : 26362). video_out: throwing away image with pts 12961625 because it's too old (diff : 54743). video_out: throwing away image with pts 12969871 because it's too old (diff : 64692). video_out: throwing away image with pts 12978285 because it's too old (diff : 74464). Floating point exception See you |
From: zaverel <za...@fr...> - 2013-03-19 16:34:26
|
Le 09/01/2013 17:37, zaverel a écrit : > Le 07/01/2013 09:57, Petri Hintukainen a écrit : >> On ke, 2012-11-14 at 16:25 +0100, zaverel wrote: >>> Le 14/11/2012 13:02, Petri Hintukainen a écrit : >>>> Attached patch (fix1.diff) should fix this. If not, fix2.diff disables >>>> that functionality for video streams. >>>> If fix1 does not help, maybe you could provide short sample recording >>>> (something like one second should be OK) ? Or some log output with >>>> attached log.diff patch ? >>>> >>>> >>>> - Petri >>>> >>> Hello, >>> >>> fix1.diff don't work. >>> >>> fix2.diff work. >>> >>> log.diff seem to have a typo >>> >>> that don't compile with >>> this->pes_length but is ok with this->pes_leng >>> >>> >>> i attach fix1.log and a link to a sample recording >>> >>> http://dl.free.fr/jD6c3V0t6 >>> >> Thanks for the log and sample, it helped me to find the root cause of >> this problem. >> >> This should be fixed in CVS now. >> >> >> - Petri >> >> >> > > hello, > > i'm afraid that's not perfect: > > Now video is good but i have random crash between 2 and 3 mn > when i am on iptv channel and only iptv channel. > > this is the log just before the crash > > video_out: throwing away image with pts 12930689 because it's too old > (diff : 3744). > video_out: throwing away image with pts 12934352 because it's too old > (diff : 9146). > video_out: throwing away image with pts 12938012 because it's too old > (diff : 14568). > video_out: throwing away image with pts 12945682 because it's too old > (diff : 25124). > video_out: throwing away image with pts 12953558 because it's too old > (diff : 26362). > video_out: throwing away image with pts 12961625 because it's too old > (diff : 54743). > video_out: throwing away image with pts 12969871 because it's too old > (diff : 64692). > video_out: throwing away image with pts 12978285 because it's too old > (diff : 74464). > Floating point exception > > See you > > Hello, I don't know if it's safe but latest xineliboutput + your patch fix2.diff and iptv doesn't crash anymore and there is no flood in log. So maybe patch has to be applied for everyone. By the way, what is the statut about ffmpeg and xine-lib-1.2 cvs? the latest working version is ffmpeg-1.0.5 More recent version doesn't compile or not work at all with vaapi. see you. |