[Gefixt] scraper2vdr (git) segfault beim Beenden des VDR

TheChief · 12. Mai 2014

Kannst Du da mal bitte schauen louis?

Code

Shutting down VDRBeendet
root@homeserver:/usr/src/vdr-2.1.6# *** glibc detected *** /usr/local/bin/vdr: free(): invalid pointer: 0x0000000002935358 ***
======= Backtrace: =========
/lib/x86_64-linux-gnu/libc.so.6(+0x76d76)[0x7f45de0b7d76]
/lib/x86_64-linux-gnu/libc.so.6(cfree+0x6c)[0x7f45de0bcb1c]
/usr/local/lib/vdr/libvdr-scraper2vdr.so.2.1.6(_ZN11cTVDBSeriesD1Ev+0x44e)[0x7f45d7518fce]
/usr/local/lib/vdr/libvdr-scraper2vdr.so.2.1.6(_ZN11cTVDBSeriesD0Ev+0x9)[0x7f45d7519279]
/usr/local/lib/vdr/libvdr-scraper2vdr.so.2.1.6(_ZN13cScrapManagerD1Ev+0x43)[0x7f45d7516d73]
/usr/local/lib/vdr/libvdr-scraper2vdr.so.2.1.6(_ZN13cScrapManagerD0Ev+0x9)[0x7f45d7516eb9]
/usr/local/lib/vdr/libvdr-scraper2vdr.so.2.1.6(_ZN18cPluginScraper2vdrD1Ev+0x35)[0x7f45d7505165]
/usr/local/lib/vdr/libvdr-scraper2vdr.so.2.1.6(_ZN18cPluginScraper2vdrD0Ev+0x9)[0x7f45d75051e9]
/usr/local/bin/vdr(_ZN4cDllD1Ev+0x26)[0x4de026]
/usr/local/bin/vdr(_ZN4cDllD0Ev+0x9)[0x4de079]
/usr/local/bin/vdr(_ZN9cListBase3DelEP11cListObjectb+0x43)[0x51fd93]
/usr/local/bin/vdr(_ZN14cPluginManager8ShutdownEb+0x61)[0x4df391]
/usr/local/bin/vdr(main+0x118b)[0x47822b]
/lib/x86_64-linux-gnu/libc.so.6(__libc_start_main+0xfd)[0x7f45de05fead]
/usr/local/bin/vdr[0x47aec5]
======= Memory map: ========

Alles anzeigen

_Martin_ · 12. Mai 2014

Habe das Problem glaub ich schon seit längerem aber da es nur beim Beenden auftritt hat es mich nie gestört.

Hier mal ein Backtrack von mir

Code

[Thread debugging using libthread_db enabled]
Using host libthread_db library "/lib/x86_64-linux-gnu/libthread_db.so.1".
Core was generated by `vdr --grab=/tmp/ -s /home/mts/Videos/vdr/kill.sh -l 3 6 -Psofthddevice -v vdpau'.
Program terminated with signal SIGABRT, Aborted.
#0  0x00007f7c7c224f79 in __GI_raise (sig=sig@entry=6) at ../nptl/sysdeps/unix/sysv/linux/raise.c:56
56	../nptl/sysdeps/unix/sysv/linux/raise.c: Datei oder Verzeichnis nicht gefunden.
(gdb) bt
#0  0x00007f7c7c224f79 in __GI_raise (sig=sig@entry=6) at ../nptl/sysdeps/unix/sysv/linux/raise.c:56
#1  0x00007f7c7c228388 in __GI_abort () at abort.c:89
#2  0x00007f7c7c2621d4 in __libc_message (do_abort=do_abort@entry=1, 
    fmt=fmt@entry=0x7f7c7c370a10 "*** Error in `%s': %s: 0x%s ***\n") at ../sysdeps/posix/libc_fatal.c:175
#3  0x00007f7c7c26cf37 in malloc_printerr (action=<optimized out>, str=0x7f7c7c370d90 "munmap_chunk(): invalid pointer", 
    ptr=<optimized out>) at malloc.c:4996
#4  0x00007f7c6b924c96 in _M_dispose (__a=..., this=<optimized out>) at /usr/include/c++/4.8/bits/basic_string.h:249
#5  ~basic_string (this=0x7f7c348ff930, __in_chrg=<optimized out>) at /usr/include/c++/4.8/bits/basic_string.h:539
#6  ~cTVDBMedia (this=0x7f7c348ff930, __in_chrg=<optimized out>) at tvdbseries.h:41
#7  cTVDBSeries::~cTVDBSeries (this=0x7f7c34900d00, __in_chrg=<optimized out>) at tvdbseries.c:42
#8  0x00007f7c6b9252d9 in cTVDBSeries::~cTVDBSeries (this=0x7f7c34900d00, __in_chrg=<optimized out>) at tvdbseries.c:50
#9  0x00007f7c6b920a7f in cScrapManager::~cScrapManager (this=0x22934b0, __in_chrg=<optimized out>) at scrapmanager.c:33
#10 0x00007f7c6b920bd9 in cScrapManager::~cScrapManager (this=0x22934b0, __in_chrg=<optimized out>) at scrapmanager.c:41
#11 0x00007f7c6b90de05 in cPluginScraper2vdr::~cPluginScraper2vdr (this=0x20fd4a0, __in_chrg=<optimized out>)
    at scraper2vdr.c:87
#12 0x00007f7c6b90de89 in cPluginScraper2vdr::~cPluginScraper2vdr (this=0x20fd4a0, __in_chrg=<optimized out>)
    at scraper2vdr.c:89
#13 0x00000000004ce8a6 in cDll::~cDll (this=0x1fff560, __in_chrg=<optimized out>) at plugin.c:189
#14 0x00000000004ce8f9 in cDll::~cDll (this=0x1fff560, __in_chrg=<optimized out>) at plugin.c:194
#15 0x000000000050db14 in cListBase::Del (this=this@entry=0x7fff924df350, Object=Object@entry=0x1fff560, 
    DeleteObject=DeleteObject@entry=true) at tools.c:2054
#16 0x00000000004cfc51 in cPluginManager::Shutdown (this=this@entry=0x7fff924df330, Log=Log@entry=true) at plugin.c:513
#17 0x000000000046a925 in main (argc=<optimized out>, argv=<optimized out>) at vdr.c:1453
(gdb)

Alles anzeigen

Grüße
Martin

louis · 13. Mai 2014

Moin,

das Problem kenne ich schon von 3P0, habe aber noch keine Lösung dafür...

Tritt der Segfault reproduzierbar beim Beenden auf? Oder nur sporadisch?

Der Crash tritt beim Aufruf des Destruktors eines cTVDBMedia Objekts auf, der Destruktor ist aber leer, das Objekt besteht neben primitiven Datentypen nur aus einem std::string, der aber initialisiert wurde. Beim Löschen dieses Strings knallt es

Ciao Louis

TheChief · 13. Mai 2014

Reproduzierbar weiss ich jetzt nicht, aber zumindest öfter. Mir war es nur aufgefallen, weil ich gerade auf meinem Server öfters VDR neustarten muss, um Tests am epg2vdr Plugin zu machen.

_Martin_ · 13. Mai 2014

Mir fällt es auch nur bei meinem Entwicklungssystem auf, da ich diesen immer wieder starte zum testen. Ob nun immer weiß ich nicht genau aber ziemlich oft, dass muss ich mal weiter beobachten.

Grüße
Martin

_Martin_ · 13. Mai 2014

Hi Louis,

könnte es evtl. hierran liegen:

Code

case msBanner3:
            banners.push_back(media);
        case msSeasonPoster:
            seasonPosters.insert(pair<int, cTVDBMedia*>(season, media));
            break;

Das ist in Datei tvdbseries.c

Da fehlt doch ein break bei den Bannern oder nicht? Wenn nun ein Banner hinzugefügt wird, wird es auch nochmal bei Season hinzugefügt. Beim Destructor werden dann erst die Banner gelöscht womit der Speicher schon frei ist, und danach werden die Seasons gelöscht und es knallt.

Sollte ja sicherlich auch logisch falsch sein das ein Banner nochmal als Season eingetragen wird.

Grüße
Martin

louis · 13. Mai 2014

Hey, das schaut gut aus! Und erklärt auch den Crash...das wirds wohl sein, du bist mein Held

Ciao Louis

louis · 14. Mai 2014

Moin,

der von Martin vorgeschlagene Fix ist jetzt im Git...damit sollten die Crashes beim Beenden nun hoffentlich weg sein.

Ciao Louis

TheChief · 14. Mai 2014

Super, dann setz ich das mal auf gelöst.

[Gefixt] scraper2vdr (git) segfault beim Beenden des VDR

Jetzt mitmachen!

Teilen