[softhddevice] SegFault bei Sprung zu Schnittmarke (gelöst)

dad401 · 19. April 2012

Soeben beim Abspielen einer Aufnahme und Sprung zur nächsten, von markad gesetzten, Schnittmarke, bekomme ich einen Segfault. Dieser ist für diese Aufnahme reproduzierbar (jedenfalls beim 2. gleichen Versuch). Ich werde es am Wochenende noch auf einem anderen VDR testen. Anbei jedoch das Log.

SoftHDdevice Version: 0.5.1-GIT09d8588

Backtrace:

Code

Core was generated by `/usr/local/bin/vdr 1> /var/log/vdr/runvdr.log 2>&1 -c /etc/vdr-1.7.17 -l 3 -P'.
Program terminated with signal 11, Segmentation fault.
#0 cSoftHdDevice::StillPicture (this=0x98bd868, data=0x0, length=0) at softhddevice.cpp:1205
1205 dsyslog("[softhddev]%s: %s %p %d\n", __FUNCTION__,
(gdb) bt
#0 cSoftHdDevice::StillPicture (this=0x98bd868, data=0x0, length=0) at softhddevice.cpp:1205
#1 0x080ae2ba in cDevice::StillPicture (this=0x98bd868, Data=0xbfdfb22c "G\002", Length=0) at device.c:1400
#2 0xb67319f3 in cSoftHdDevice::StillPicture (this=0x98bd868, data=0xbfdf62dc "G@", length=20492) at softhddevice.cpp:1209
#3 0x080b81fb in cPlayer::DeviceStillPicture (this=0x9b31948, Index=10560, Still=true) at player.h:36
#4 cDvbPlayer::Goto (this=0x9b31948, Index=10560, Still=true) at dvbplayer.c:881
#5 0x080d0a3a in cReplayControl::MarkJump (this=0x914ca68, Forward=true) at menu.c:5791
#6 0x080ecc26 in cReplayControl::ProcessKey (this=0x914ca68, Key=k9) at menu.c:5978
#7 0xb36bd3f9 in myReplayControl::ProcessKey (this=0x914ca68, Key=k9) at myreplaycontrol.c:76
#8 0x0813c5d0 in main (argc=56, argv=0xbfef6654) at vdr.c:1169
Current language: auto
The current source language is "auto; currently c++".
(gdb)

Alles anzeigen

syslog:

Code

Apr 19 21:50:58 antecvdr vdr: [1933] [softhddev]SetPlayMode: 0
Apr 19 21:50:58 antecvdr vdr: [1933] [softhddev]SetVideoDisplayFormat: 2
Apr 19 21:50:58 antecvdr vdr: [1933] [extrecmenu] starting replay of recording
Apr 19 21:50:58 antecvdr vdr: [1933] replay /video0/Two_and_a_Half_Men/Ein_Sarg_ist_keine_Sonnenbank/2012-04-17.21.05.7-0.rec
Apr 19 21:50:58 antecvdr vdr: [1933] playing '/video0/Two_and_a_Half_Men/Ein_Sarg_ist_keine_Sonnenbank/2012-04-17.21.05.7-0.rec/00001.ts'
Apr 19 21:50:58 antecvdr vdr: [1933] loading /video0/Two_and_a_Half_Men/Ein_Sarg_ist_keine_Sonnenbank/2012-04-17.21.05.7-0.rec//marks
Apr 19 21:50:58 antecvdr vdr: [2369] EnigmaNG effects thread ended (pid=1933, tid=2369)
Apr 19 21:50:58 antecvdr vdr: [1933] [softhddev]SetPlayMode: 1
Apr 19 21:50:58 antecvdr vdr: [2374] dvbplayer thread started (pid=1933, tid=2374)
Apr 19 21:50:58 antecvdr vdr: [2375] non blocking file reader thread started (pid=1933, tid=2375)
Apr 19 21:50:58 antecvdr vdr: video: decoder buffer empty, duping frame (739/298598) 0 v-buf
Apr 19 21:50:58 antecvdr vdr: video: 0:22:07.536 -11 0 240/\ms 0 v-buf
Apr 19 21:50:58 antecvdr vdr: audio/alsa: wait underrun error? 'Datenübergabe unterbrochen (broken pipe)'
Apr 19 21:50:58 antecvdr vdr: audio/alsa: using device 'default'
Apr 19 21:50:58 antecvdr vdr: audio/alsa: delay 264ms
Apr 19 21:50:59 antecvdr vdr: video: decoder buffer empty, duping frame (744/298598) 1 v-buf
Apr 19 21:50:59 antecvdr vdr: video/vdpau: missed frame (6/10)
Apr 19 21:50:59 antecvdr vdr: [2376] EnigmaNG effects thread started (pid=1933, tid=2376)
Apr 19 21:51:00 antecvdr vdr: [1968] EPGSearch: timer conflict check started
...
Apr 19 21:51:00 antecvdr vdr: [1968] EPGSearch: timer conflict check finished
Apr 19 21:51:01 antecvdr vdr: [1933] [softhddev]Clear:
Apr 19 21:51:01 antecvdr vdr: video: decoder buffer empty, duping frame (766/108) 0 v-buf
Apr 19 21:51:01 antecvdr vdr: video: 3:11:49.038+8888 0 240/\ms 0 v-buf
Apr 19 21:51:01 antecvdr vdr: [1933] [softhddev]StillPicture: ts 0xbfdf62dc 20492
Apr 19 21:51:01 antecvdr kernel: [ 7219.076667] vdr[1933]: segfault at 0 ip b673197c sp bfdf61b0 error 4 in libvdr-softhddevice.so.1.7.17[b6726000+27000]
Apr 19 21:51:02 antecvdr lircd-0.8.7[1789]: read() failed
Apr 19 21:51:02 antecvdr lircd-0.8.7[1789]: Connection reset by peer
Apr 19 21:51:02 antecvdr lircd-0.8.7[1789]: removed client
Apr 19 21:51:02 antecvdr kernel: [ 7219.741883] display port closed
Apr 19 21:51:04 antecvdr smartd[1687]: Device: /dev/sdb [SAT], SMART Usage Attribute: 194 Temperature_Celsius changed from 113 to 112

Alles anzeigen

Ein bekannter, oder ggf. neuer Bug? Komme leider noch nicht dazu eine neue GIT-Version zu testen. Interesse an der Aufnahme?

Gruss
Marcus

johns · 19. April 2012

Naja am Plugin liegt es nur, wenn NULL Pointer erlaubt sind.

Code

cSoftHdDevice::StillPicture (this=0x98bd868, data=0x0, length=0)

Da ruft jemand StillPicture mit NULL Pointer auf. Das knallt dann sofort und immer.

Johns

dad401 · 20. April 2012

Zitat von johns

Da ruft jemand StillPicture mit NULL Pointer auf. Das knallt dann sofort und immer.

Da kann ja in diesem Fall nur der VDR sein, oder? Müsste man dann dort bereits NULL Pointer abfangen? - ich kann das ganze auch mal mit xine testen.

Marcus

johns · 20. April 2012

Kannst vorerst erstmal ein "if (!data) return;" in cSoftHdDevice::StillPicture machen.

Da es bei meinen VDR nicht vorkommt, könnte es auch an einem Patch liegen.
Bei vdr-xineliboutput sollte es auch knallen, da habe ich geguckt ob die dies abfragen.

Johns

FireFly · 20. April 2012

Mir kommen diese EInträge komisch vor:

Code: index.php?page=Thread&postID=1066799#post1066799

#0 cSoftHdDevice::StillPicture (this=0x98bd868, data=0x0, length=0) at softhddevice.cpp:1205
#1 0x080ae2ba in cDevice::StillPicture (this=0x98bd868, Data=0xbfdfb22c "G\002", Length=0) at device.c:1400
#2 0xb67319f3 in cSoftHdDevice::StillPicture (this=0x98bd868, data=0xbfdf62dc "G@", length=20492) at softhddevice.cpp:1209
#3 0x080b81fb in cPlayer::DeviceStillPicture (this=0x9b31948, Index=10560, Still=true) at player.h:36

Da ruft cPlayer:: DeviceStillPicture das cSoftHdDevice:: StillPicture auf und dort ist noch length=20492, im nächsten cDevice:: StillPicture ist Length=0, was dann im nachfolgenden cSoftHdDevice:: StillPicture crasht.

gda · 20. April 2012

Was ruft denn überhaupt DeviceStillPicture auf? Die Fernbedienung im Live-Plugin?

Gerald

johns · 20. April 2012

Zitat von FireFly
Mir kommen diese EInträge komisch vor:
Code: index.php?page=Thread&postID=1066799#post1066799
#0 cSoftHdDevice::StillPicture (this=0x98bd868, data=0x0, length=0) at softhddevice.cpp:1205
#1 0x080ae2ba in cDevice::StillPicture (this=0x98bd868, Data=0xbfdfb22c "G\002", Length=0) at device.c:1400
#2 0xb67319f3 in cSoftHdDevice::StillPicture (this=0x98bd868, data=0xbfdf62dc "G@", length=20492) at softhddevice.cpp:1209
#3 0x080b81fb in cPlayer::DeviceStillPicture (this=0x9b31948, Index=10560, Still=true) at player.h:36
Da ruft cPlayer:: DeviceStillPicture das cSoftHdDevice:: StillPicture auf und dort ist noch length=20492, im nächsten cDevice:: StillPicture ist Length=0, was dann im nachfolgenden cSoftHdDevice:: StillPicture crasht.

Ist schon rightig, wenn ein TS (Transport Stream) Paket kommt, dann kann dies durch VDR nach PES gewandelt werden.

Zitat

///< Data points either to TS (first byte is 0x47) or PES (first byte
///< is 0x00) data of the given Length. The default implementation
///< converts TS to PES and calls itself again, allowing a derived class
///< to display PES if it can't handle TS directly.

FireFly · 20. April 2012

Zitat von gda

Was ruft denn überhaupt DeviceStillPicture auf? Die Fernbedienung im Live-Plugin?

Lt. Backtrace von oben cDvbPlayer::Goto (this=0x9b31948, Index=10560, Still=true) und das ist ja auch korrekt (egal von welcher FB und durch welchen Tastendruck auch immer das ausgelöst wurde)

FireFly · 20. April 2012

Zitat von johns

Ist schon rightig, wenn ein TS (Transport Stream) Paket kommt, dann kann dies durch VDR nach PES gewandelt werden.

Ja, schon klar, aber warum ist die Länge nachher Null? Das würde ja bedeuten, dass alles im TS-Paket verworfen wurde. Und mit Null Daten crasht er natürlich.

dad401 · 22. April 2012

Ich habe nun xine/xine-ui und xineliboutput getestet - hier passiert kein Absturz. Stattdessen wird zur Schnittmarke gesprungen, wobei das angezeigte Bild dem letzten Standbild vor dem Sprung entspricht.

Ich vermute, dass es sich um eine von MarkAd falsch gesetzte Schnittmarke handelt (kein I-Frame?) - dennoch sollte der VDr nicht crashen. Problem wurde an einem weiteren System bestätigt.

Marcus

johns · 22. April 2012

Die haben einen eigenen Parser.

Es scheint bei der Wandlung von TS -> PES zupassieren. Schreib es mal Klaus.

Johns

kls · 26. April 2012

Zitat von johns

Die haben einen eigenen Parser.

Es scheint bei der Wandlung von TS -> PES zupassieren. Schreib es mal Klaus.

Ich hab's mal getestet und es wird tatsächlich StillPicture() mit NULL aufgerufen.
Offensichtlich findet cDevice::StillPicture() keine brauchbaren Daten und legt daher auch keinen Puffer an.
Anscheinend fangen das manche Devices ab, andere nicht. Da nirgens steht, daß Data == NULL sein kann, kann man einem Device, welches das nicht abfrägt, auch keinen Vorwurf machen

Folgende Änderung sollte das fixen:

Diff

--- device.c    2012/03/31 11:13:16     2.59
+++ device.c    2012/04/26 09:40:36
@@ -1181,8 +1181,10 @@
               return;
               }
            }
-     StillPicture(buf, Size);
-     free(buf);
+     if (buf) {
+        StillPicture(buf, Size);
+        free(buf);
+        }
      }
 }

Alles anzeigen

Ich schätze auch, daß das Problem von einer falsch gesetzten Schnittmarke kommt.
Wenn man die erste Marke in der Testaufnahme anspringt, dann wird dort (mit einer TT S2-6400, bei der kein Absturz erfolgt, weil auf Data!=NULL geprüft wird) kein passendes Standbild angezeigt. Verschiebt man aber die Marke, dann werden passende Standbilder angezeigt.

Klaus

[softhddevice] SegFault bei Sprung zu Schnittmarke (gelöst)

Jetzt mitmachen!

Teilen

Benutzer online in diesem Thema