[BUG] Segfault movieplayer/Infoviewer

[GetAway]

Hi,

bei folgender Tastenreihenfolge bekomme ich einen Segfault.
Filmarchiv öffnen.
Film abspielen.
Mit der OK-Taste Filmarchiv, während des abspielens, erneut öffnen/schließen.
2x Help-Taste drücken.

Der Fehler trit nur auf, wenn in den Filmarchiv Einstellungen
"Filminfos neu Laden bei Start" aktiv ist. Lädt man die Filminfos
über die Blaue Taste neu, tritt der Fehler nicht auf.
Sollte einfach zu fixen sein, wenns einer machen will.

[Gaucho316]

Hmm, eigentlich müsste das Problem immer auftreten, egal, ob das Neuladen automatisch oder von Hand passiert. Dem Movieplayer wird nämlich der Pointer p_movie_info unterm Hintern weggeklaut, wenn ich das richtig überblicke.

[GetAway]

Ja, ist Glückssache, wenn es einem nicht um die Ohren fliegt.

Tausche mal im mp die Zeile 4155 gegen ..printDebugMovieInfo(..) und
schau dir den Audiopid Ausgabe an. Mach aber vorher in movieinfo.cpp 670
aus %d ein %llu, sonst fliegt es dir auch dort um die Ohren.

[Gaucho316]

Wie wäre es damit, die blaue und die d-box Taste sowie das automatische Neuladen einfach abzuschalten, wenn man den Moviebrowser während des Abspielens öffnet? Dann hat man keine Möglichkeit mehr, das Neuladen auszulösen. Ich habe schon einen Patch in der Hinterhand, muss aber nochmal drüber schlafen.

[GetAway]

Das neutralisiert vielleicht die Auswirkung, löst aber nicht den Bug, der da irgendwo drinsteckt.
Ich habe es mal markiert.

Code: Alles auswählen

 dateOfLastPlay:        1383689403
 dirItNr:               0
 genreMajor:            21
 genreMinor:            0
 length:                50
 length rec:            11
 quality:               0
 productionCount:       ><
 productionDate:        0
 parentalLockAge:                     16
 format:                0
 audio:                 0
 epgId:                 8716324
 epgEpgId:              4785645837104473
 epgMode:               1
 epgVideoPid:           1023
 epgVTXPID:             0
 Size:          4
 Date:          1383588404
 audioPid (1162363595070111744):        -858993459           <-------------------
 audioName(0):          >Deutsch<
 epgTitle:              >Battlestar Galactica<
 epgInfo1:              >Virus<
 epgChannel:            >Syfy<
 serieName:             ><
 bookmarks start:       0
 bookmarks end:         0
 bookmarks lastPlayStop: 8
 bookmarks user, pos:0, type:0, name: ><

[dbt]

Wirf mal den Compiler mit -Wconversion an, würde mich nicht wundern, wenn dann das eine oder andere Licht aufgeht. Bin mir jetzt aber nicht sicher, ob das default an ist, aber dann müsste man ja schon was sehen.

[GetAway]

Welche Stelle muss ich wo ändern?

Wahrscheinlich configure.ac im Neutrino-Verzeichnis?

[dbt]

Welche Stelle muss ich wo ändern?

Gib mal im CDK Ordner ein:

Code: Alles auswählen

git grep TARGET_CXXFLAGS

Da müssten relevante Stellen zum Vorschein kommen, wo solche flags hinkommen könnten.
Infos zu -Wconversion und noch mehr sollten hier zu finden sein, kommt auch auf den Compiler an, den du nutzt, da gibts auch 'ne Liste.

[Gaucho316]

Das neutralisiert vielleicht die Auswirkung, löst aber nicht den Bug, der da irgendwo drinsteckt.

Das Problem ist doch, dass du auf nicht mehr zugewiesenen Speicher zugreifst, oder nicht? Das kann doch nur gegen den Baum laufen.

[GetAway]

Dein Vorschlag von oben ist wohl die einfachste Lösung. Das der Speicher nicht mehr zugewiesen ist,
habe ich gar nicht gesehen.

[Gaucho316]

Ich hab jetzt mal einen Patch gebastelt.