[gelöst] tvscraper 0.0.1 Absturz bei scrappen des Videoverzeichnis

Zitat von Taipan

Kann man anhand dessen sagen worans liegt?

Schlechter Geschmack bei den Suchtimern?
Die Pakete für vdr-plugin-tvscrapper, die bald im testing-PPA bereit stehen sollten bringen mit dem Paket vdr-plugin-tvscrapper-dbg Debug-Symbole mit - damit solltest du wie bei allen anderen VDR-Crashes auch mit den gewohnten Methoden auf die Suche gehen können.

Hi,

jo...nen Backtrace wäre sehr hilfreich...das Paket mit den Debug Symbolen ist ja schon gebaut

Mit "Alles was zählt" direkt hat das wohl nichts zu tun...das wird ja erfolgreich gescrapt. Was würde denn danach kommen? Da scheint mir eher das Problem zu liegen...aber das ist nur Raterei, ich warte mal lieber auf den Backtrace.

Ciao Louis

Hmm, die Zeile selbst sieht ziemlich harmlos aus:

string recTitle = recInfo->Title();

Allerdings gibt recInfo->Title() keinen string zurück, sondern ein const char * und beim cast geht was schief.
Keine Ahnung warum. Noch mehr irritierend ist aber, dass recTitle anscheinend gar nicht gebraucht wird.
Kommentiere doch die Zeile 157 in worker.c einfach mal aus.

Gerald

Zitat von Taipan

Habe die Zeile eliminiert - damit läuft es durch (jetzt wieder mit allen Aufnahmen) und liefert auch Daten/Bilder...

Na dann kannst du ja jetzt einen Bugreport machen.

Gerald

Moin,

Gerald hat Recht...die Zeile ist wirklich überflüssig. Wenn dadurch der Crash behoben ist, ist es ja gut, anscheinend wird hier in speziellen Fällen (wie bei Taipan) auf eine Nullreferenz zugegriffen, das mag wohl std::string als Initialisierung nicht. Ich werde das in der nächsten version rausschmeissen.

Ciao Louis

Zitat von louis

anscheinend wird hier in speziellen Fällen (wie bei Taipan) auf eine Nullreferenz zugegriffen, das mag wohl std::string als Initialisierung nicht.

Meine Vermutung geht eher in Richtung UTF8-Probleme. Solange du mit einen UTF8-String in einem Char-Array nichts machst, sondern höchstens herumkopierst, dann geht alles gut. Aber wenn der String plötzlich interpretiert wird beim Cast, dann geht das wohl in die Hose.
Ist jetzt aber nur ein wild guess.

Gerald

Man, fast 50 Threads durchsuchen nur um dann festzustellen, dass das der Fehler ist den ich gestern schon gefixt hatte. Das ist nicht dein Ernst oder? Ansonsten poste doch mal deine Zeile 157 aus worker.c.

Und das ist übrigens der Grund warum ich denke der Scraper für recordings gehört in ein eigenes exe, dann reißt er den VDR nicht runter.

Gerald

Aug 22 20:01:40 Server vdr: [10711] tvscrapper: scrapping series "Sweet & Easy - Enie backt"
Aug 22 20:01:45 Server vdr: [10711] tvscrapper: "Sweet & Easy - Enie backt" successfully scrapped, id 266707
Aug 22 20:01:45 Server vdr: [10711] tvscrapper: scrapping series "Album 2005 - Bilder eines Jahres"
Aug 22 20:01:46 Server vdr: [10711] tvscrapper: nothing found for "Album 2005 - Bilder eines Jahres"
Aug 22 20:01:46 Server vdr: [10711] tvscrapper: scrapping series "Album 2004 - Bilder eines Jahres"
Aug 22 20:01:46 Server vdr: [10711] tvscrapper: nothing found for "Album 2004 - Bilder eines Jahres"
Aug 22 20:01:46 Server vdr: [10711] tvscrapper: scrapping series "Album 2001 - Bilder eines Jahres"
Aug 22 20:01:46 Server vdr: [10711] tvscrapper: nothing found for "Album 2001 - Bilder eines Jahres"
Aug 22 20:01:46 Server vdr: [10711] tvscrapper: query failed: select series_id from series where series_name='Album '90 - Bilder eines Jahres' , error: near "90": syntax error
Aug 22 20:01:46 Server vdr: [10711] tvscrapper: scrapping series "Album '90 - Bilder eines Jahres"
Aug 22 20:01:47 Server vdr: [10711] tvscrapper: nothing found for "Album '90 - Bilder eines Jahres"
Aug 22 20:01:47 Server vdr: [10711] tvscrapper: query failed: select series_id from series where series_name='Album '82 - Bilder eines Jahres' , error: near "82": syntax error
Aug 22 20:01:47 Server vdr: [10711] tvscrapper: scrapping series "Album '82 - Bilder eines Jahres"
Aug 22 20:01:47 Server vdr: [10711] tvscrapper: nothing found for "Album '82 - Bilder eines Jahres"
Aug 22 20:01:47 Server kernel: [ 7652.292253] init: vdr main process (10580) killed by ABRT signal

hier bei mir ist eindeutig das ' im Namen schuld.
nach ändern der jeweiligen info Datei läufts

PS. in worker.c habe ich die Zeile 157 auskommentiert und Plugin neu compiliert

Zitat von machtnix

PS. in worker.c habe ich die Zeile 157 auskommentiert und Plugin neu compiliert

Dann fehlt aber jetzt die Info welche Zeile er statt der Zeile 157 anmeckert.

Gerald

Zitat von machtnix

Code

Aug 22 20:01:40 Server vdr: [10711] tvscrapper: scrapping series "Sweet & Easy - Enie backt"
Aug 22 20:01:45 Server vdr: [10711] tvscrapper: "Sweet & Easy - Enie backt" successfully scrapped, id 266707
Aug 22 20:01:45 Server vdr: [10711] tvscrapper: scrapping series "Album 2005 - Bilder eines Jahres"
Aug 22 20:01:46 Server vdr: [10711] tvscrapper: nothing found for "Album 2005 - Bilder eines Jahres"
Aug 22 20:01:46 Server vdr: [10711] tvscrapper: scrapping series "Album 2004 - Bilder eines Jahres"
Aug 22 20:01:46 Server vdr: [10711] tvscrapper: nothing found for "Album 2004 - Bilder eines Jahres"
Aug 22 20:01:46 Server vdr: [10711] tvscrapper: scrapping series "Album 2001 - Bilder eines Jahres"
Aug 22 20:01:46 Server vdr: [10711] tvscrapper: nothing found for "Album 2001 - Bilder eines Jahres"
Aug 22 20:01:46 Server vdr: [10711] tvscrapper: query failed: select series_id from series where series_name='Album '90 - Bilder eines Jahres' , error: near "90": syntax error
Aug 22 20:01:46 Server vdr: [10711] tvscrapper: scrapping series "Album '90 - Bilder eines Jahres"
Aug 22 20:01:47 Server vdr: [10711] tvscrapper: nothing found for "Album '90 - Bilder eines Jahres"
Aug 22 20:01:47 Server vdr: [10711] tvscrapper: query failed: select series_id from series where series_name='Album '82 - Bilder eines Jahres' , error: near "82": syntax error
Aug 22 20:01:47 Server vdr: [10711] tvscrapper: scrapping series "Album '82 - Bilder eines Jahres"
Aug 22 20:01:47 Server vdr: [10711] tvscrapper: nothing found for "Album '82 - Bilder eines Jahres"
Aug 22 20:01:47 Server kernel: [ 7652.292253] init: vdr main process (10580) killed by ABRT signal

Alles anzeigen

hier bei mir ist eindeutig das ' im Namen schuld.
nach ändern der jeweiligen info Datei läufts

PS. in worker.c habe ich die Zeile 157 auskommentiert und Plugin neu compiliert

Da muss wohl das Apostroph im Statement noch escaped werden. Gibts bestimmt ne Funktion für - eventuell passiert das auch bei anderen Zeichen.

Zitat von TheChief

Da muss wohl das Apostroph im Statement noch escaped werden. Gibts bestimmt ne Funktion für - eventuell passiert das auch bei anderen Zeichen.

Das hängt doch von der Zeile ab in der das passiert. Das ist doch kein Shellscript. Da ist eher die Frage ob da iconv() oder ähnliches benutzt werden muss.

Gerald

Naja, auch in C muss sowas doch escaped werden. Bin kein Experte. In php gibts da beispielsweise mysql_real_escape_string für sql abfragen. Und ja ich weiss, es ist sqlite. Sieht man doch auch schön im Logsauzug.

Zitat von TheChief

Naja, auch in C muss sowas doch escaped werden.

Nein, doch nicht bei einer Zuweisung zweier Variablen. Außerdem würde Louis den ganzen Problemen aus dem Weg gehen, wenn er die char * einfach char * lassen würde anstatt sie zum string zu machen. Die Fehler kommen eindeutig von der Umwandlung in den string. Und schneller wäre es auch noch, ist aber nicht so wichtig.

Wenn es unbedingt eine String-Klasse sein muss, dann vielleicht die cString-Klasse von Klaus aus tools.h? Vielleicht kommt die ja besser damit klar.

Gerald

Na wenn Du meinst. Sowas geht jedenfalls nicht:

select series_id from series where series_name='Album '90 - Bilder eines Jahres'