Segmentation-Fault bei Internet-Update mit Squashfs-Check

[kroki]

Hi,
habe sei den heutigen Änderungen beim Flash-Update, ein Absturz der Box nach der Bestätigung der Version. Der Download nicht gestartet.

Gruß Kroki

[Nachtvogel]

Hallo!

Was hast Du denn für ein Image auf der Box?
Bei Yadi funktioniert das Internetupdate mit der Box nicht.

Gruß Nachtvogel

[JtG-Riker]

Ist schon bekannt der Fehler, denke Mogway wird dabei nochmal schauen.

[kroki]

Ich habe mein eigenes Image auf meinen Boxen ! Und ich mache das Internet-Update über meinen eigenen Server....


Danke für die Info JTG-Riker...Ich war auch schon am suchen, bin aber noch nicht fündig geworden !

Kroki

[JtG-Riker]

Naja fündig geworden bin ich schon, allerdings wenn man es fixt geht das manuelle Update nicht mehr richtig, da blick ich irgendwie nicht durch.

Riker

[siamtec]

Ich kann das nur bestätigen. Ich nutze auch die Updatefunktion über meinen Server.

Nachdem ich heute ein Update vom CVS gemacht habe und dabei auch die update.cpp aktualisiert und die checksquashfs.cpp eingebunden habe, geht nichts mehr.

Er zeigt mir noch die Versionen an, die bei mir auf dem Server liegen aber nach Auswahl einer Version hängt sich meine Dbox auf.

Ich habe mal zum testen die checksquashfs.cpp und die update.cpp wieder auf alten Stand zurückgesetzt. Danach ging alles wieder.

Habt Ihr da schon einen Bug gefunden?

[JtG-Riker]

Jo, muss nur ma noch jemand ändern

[siamtec]

Jo, muss nur ma noch jemand ändern

Sehr schön.

Kannst Du die Änderung mal bitte posten? Dann kann ich das bei mir per Hand ändern.

LG
siamtec

[JtG-Riker]

Code: Alles auswählen

Index: update.cpp
===================================================================
RCS file: /cvs/tuxbox/apps/tuxbox/neutrino/src/gui/update.cpp,v
retrieving revision 1.113
diff -u -r1.113 update.cpp
--- a/update.cpp	6 Oct 2005 22:05:27 -0000	1.113
+++ b/update.cpp	7 Oct 2005 21:51:05 -0000
@@ -255,6 +255,7 @@
 		versionInfo = new CFlashVersionInfo(newVersion);
 
 		msg_body = LOCALE_FLASHUPDATE_MSGBOX;
+/*
 #ifdef SQUASHFS
 		sprintf(msg, g_Locale->getText(msg_body), versionInfo->getDate(), versionInfo->getTime(), versionInfo->getReleaseCycle(), versionInfo->getType());
 
@@ -274,6 +275,7 @@
 
 		delete versionInfo;
 #endif
+*/
 	}
 	else
 	{

[JtG-Riker]

rasc hats schon gerichtet

[siamtec]

Danke, hat funktioniert

[Barf]

ok, ich beziehe mich nicht auf das Problem des ersten Posters, finde ich muss es trotzdem sagen:

Es kann nicht sinnvoll sein, dass ein High-level-programm wie Neutrino unterschiedlich kompiliert wird, abhängig von dem (low-level) Filesystem!! Etwas besser, aber trotzdem keinerfalls gut, wäre es, Präprozessorsymbolen wie ENABLE_CRAMFS_FLASHING btw ENABLE_SQUASHFS_FLASHING/ENABLE_JFFS2FS_FLASHING zu benutzen. Einzig Vernünftige wäre solche bedingte Kompilierung zu entfernen.

[rasc]

mein Reden...

... wenn unbedingt dann sollte das in ein Modul versteckt werden, dass man auch komplett weglassen kann, oder eben bei Bedarf dazukompiliert.

Aber in neutrino wird halt jedes Hündchen "irgendwie" implementiert...

Das ganze Update haette man nicht in Neutrino implementieren sollen, sondern besser als Plugin... so kann man die Update-Methoden (JTG, Yadi, Schlag-mich-tot) einfach austauschen und hat das Zeugs nicht irgendwo versteckt hart in der GUI codiert.

... aber auch mich hört sowieso keiner (so genug gemeckert)