[BUG] Sectionsd Bug ?

cvs diff: Diffing .                                                             
Index: dmxapi.cpp
===================================================================
RCS file: /cvs/tuxbox/apps/tuxbox/neutrino/daemons/sectionsd/dmxapi.cpp,v
retrieving revision 1.5
diff -u -r1.5 dmxapi.cpp
--- a/dmxapi.cpp        13 Jan 2005 10:48:02 -0000      1.5
+++ b/dmxapi.cpp        19 Feb 2009 10:52:00 -0000
@@ -116,6 +116,7 @@
        {
                printf("[sectionsd] getUTC: invalid %s section received: %02x %02x %02x %02x %02x\n",
                        TDT ? "TDT" : "TOT", cUTC[0], cUTC[1], cUTC[2], cUTC[3], cUTC[4]);
+               ::close(fd);
                return false;
        }

Index: sectionsd.cpp
===================================================================
RCS file: /cvs/tuxbox/apps/tuxbox/neutrino/daemons/sectionsd/sectionsd.cpp,v
retrieving revision 1.286
diff -u -r1.286 sectionsd.cpp
--- a/sectionsd.cpp     24 Jan 2009 17:34:45 -0000      1.286
+++ b/sectionsd.cpp     19 Feb 2009 10:52:03 -0000
@@ -2143,10 +2143,9 @@
                                s->second->eitScheduleFlag(), s->second->eitPresentFollowingFlag(),
                                s->second->runningStatus(), s->second->freeCAmode(),
                                s->second->nvods.size());
-/**    soll es in count ?
-                       + strlen(s->second->serviceName.c_str()) + 1
-                       + strlen(s->second->providerName.c_str()) + 1
-                       + 3;  **/
+
+               count += s->second->serviceName.length()+s->second->providerName.length()+1+1+3;
+
                if (count < MAX_SIZE_SERVICELIST)
                {
                        strcat(serviceList, daten);

Beitrag von **seife** » Donnerstag 19. Februar 2009, 14:06

Den ersten Teil habe ich committed. Beim zweiten Teil wird zweifelsohne jetzt falsch gezählt.
Ich behaupte aber, du zählst auch nicht richtig:

re_Look hat geschrieben:

Code: Alles auswählen

Index: sectionsd.cpp
===================================================================
RCS file: /cvs/tuxbox/apps/tuxbox/neutrino/daemons/sectionsd/sectionsd.cpp,v
retrieving revision 1.286
diff -u -r1.286 sectionsd.cpp
--- a/sectionsd.cpp     24 Jan 2009 17:34:45 -0000      1.286
+++ b/sectionsd.cpp     19 Feb 2009 10:52:03 -0000
@@ -2143,10 +2143,9 @@
                                s->second->eitScheduleFlag(), s->second->eitPresentFollowingFlag(),
                                s->second->runningStatus(), s->second->freeCAmode(),
                                s->second->nvods.size());
-/**    soll es in count ?
-                       + strlen(s->second->serviceName.c_str()) + 1
-                       + strlen(s->second->providerName.c_str()) + 1
-                       + 3;  **/
+
+               count += s->second->serviceName.length()+s->second->providerName.length()+1+1+3;
+
                if (count < MAX_SIZE_SERVICELIST)
                {
                        strcat(serviceList, daten);

Wozu "+1+1"? (Ich behaupte weiter oben das "count += 1 + ..." ist auch schon falsch).

Wozu "+1+1"?
Weil 2 mal "strcat(serviceList, "\n");" da ist.
Naja ob ich 100% zähle, weiss ich auch nicht

, aber so wie jetzt ist, kann es segfaulten.

Beitrag von **seife** » Donnerstag 19. Februar 2009, 14:34

Das "\n" wird 3 mal angehängt. Dafür ist die +3. Aber wofür sind die "+1"?
Ich sage, das sind "Angstbytes"

Ich vermute, dass da mal jemand Platz für die '\0' am String-Ende gelassen hat, die tatsächlich aber gar nicht drangehangen werden.

Jo, hast recht.
Apropos '\0' , wieso wird es nirgendwo in sectionsd am Ende dran gehängt ?

Beitrag von **seife** » Donnerstag 19. Februar 2009, 14:53

Ich sagte nicht "nirgends". Aber in diesem Fall hängt halt nur ganz hinten einmal ein \0 dran, und nicht nach jedem strcat "in der Mitte", so wie anscheinend gezählt wurde.

Ich meine nicht bei jeden strcat bzw in schleife sonder, danach vor "msgResponse.dataLength = strlen(serviceList) + 1;" .

Beitrag von **seife** » Donnerstag 19. Februar 2009, 16:18

Das ist IMHO schon korrekt. Aber ich werde es mir mal genauer anschauen.
Ist auf die TODO-Liste gepushed

Beitrag von **re_Look** » Freitag 20. Februar 2009, 15:46

Wie lange ist deine TODO Liste ?

Hier noch ein BUG bzw Fix.

Code: Alles auswählen

cvs diff: Diffing .
Index: sectionsd.cpp
===================================================================
RCS file: /cvs/tuxbox/apps/tuxbox/neutrino/daemons/sectionsd/sectionsd.cpp,v
retrieving revision 1.286
diff -u -r1.286 sectionsd.cpp
--- a/sectionsd.cpp     24 Jan 2009 17:34:45 -0000      1.286
+++ b/sectionsd.cpp     20 Feb 2009 14:42:36 -0000
@@ -4769,8 +4769,8 @@
 {
        struct sectionsd::msgResponseHeader responseHeader;
        responseHeader.dataLength = 0;
-       char * buf = (char*)malloc(64);
-       char *val = (char*)malloc(32);  // needed for SETENV?-macros
+       char * buf = new char[64];;
+       char *val = new char[32];;      // needed for SETENV?-macros
        int count;
        if (val && buf && (count = readlink("/proc/self/exe", buf, 63)) >= 0) {
                buf[count] = '\0';
@@ -4805,7 +4805,8 @@
        execv(buf, p);
        perror("[sectionsd] commandRestart execv");
        fprintf(stderr, "[sectionsd] ERROR! This is impossible!\n\n");
-       free(buf);
+       delete[] buf;
+       delete[] val;
 }

 struct s_cmd_table

Beitrag von **GetAway** » Sonntag 22. Februar 2009, 18:59

Also, wenn dies nicht nur eine Schönheitskorrektur ist, sollte es nochmal jemand
unter die Lupe nehmen und einchecken.

Beitrag von **seife** » Sonntag 22. Februar 2009, 19:49

Das letzte ist eine Schönheitskorrektur (wenn der sectionsd an der Stelle ankommt, wo er den Speicher freigibt, hast du ein anderes Problem

)

Das falsche Zählen ist mir bisher in der Praxis nicht aufgefallen (durch segfaults oder so), aber ja, man sollte das anschauen.

Beitrag von **re_Look** » Montag 2. März 2009, 14:48

seife hat geschrieben:Das letzte ist eine Schönheitskorrektur (wenn der sectionsd an der Stelle ankommt, wo er den Speicher freigibt, hast du ein anderes Problem )

Wenn fehlende free nur Schönheitskorrektur ist, dann weiß ich nicht was ein BUG ist und wenn die code nicht gebraucht wird, wieso ist die überhaupt in sectionsd gelandet ?

Beitrag von **seife** » Montag 2. März 2009, 15:33

Hinter dem execve() solltest du nie ankommen. Insofern hat du recht, das free() könnte auch noch weg.

(und ja, letzte Woche ist mein sectionsd hinter dem execve angekommen, aber da waren die paar bytes leaked memory dann auch egal

)

Beitrag von **re_Look** » Freitag 6. März 2009, 09:50

Ich hab noch ein Patch.
Wenn jemand es testen will.
Bitte Speicher verbrauch in Auge behalten.

Code: Alles auswählen

Index: SIevents.cpp
===================================================================
RCS file: /cvs/tuxbox/apps/tuxbox/neutrino/daemons/sectionsd/SIevents.cpp,v
retrieving revision 1.35
diff -u -r1.35 SIevents.cpp
--- a/SIevents.cpp	16 Aug 2008 19:23:18 -0000	1.35
+++ b/SIevents.cpp	6 Mar 2009 08:31:14 -0000
@@ -75,6 +75,7 @@
 	transport_stream_id = _transport_stream_id;
 	service_id          = _service_id;
 	eventID		    = _event_id;
+	running=0;
 /*	contentClassification = "";
 	userClassification = "";
 	itemDescription = "";
Index: SIevents.hpp
===================================================================
RCS file: /cvs/tuxbox/apps/tuxbox/neutrino/daemons/sectionsd/SIevents.hpp,v
retrieving revision 1.29
diff -u -r1.29 SIevents.hpp
--- a/SIevents.hpp	16 Aug 2008 19:23:18 -0000	1.29
+++ b/SIevents.hpp	6 Mar 2009 08:31:15 -0000
@@ -341,6 +341,7 @@
 		original_network_id = 0;
 		transport_stream_id = 0;
 		eventID    = 0;
+		running=0;
 //      dauer=0;
 //      startzeit=0;
 	}
Index: SIservices.hpp
===================================================================
RCS file: /cvs/tuxbox/apps/tuxbox/neutrino/daemons/sectionsd/SIservices.hpp,v
retrieving revision 1.15
diff -u -r1.15 SIservices.hpp
--- a/SIservices.hpp	24 Feb 2009 19:09:10 -0000	1.15
+++ b/SIservices.hpp	6 Mar 2009 08:31:15 -0000
@@ -139,6 +139,7 @@
 		transport_stream_id = _transport_stream_id;
 		serviceTyp=0;
 		memset(&flags, 0, sizeof(flags));
+		is_actual = false;
 	}
 	// Std-Copy
 	SIservice(const SIservice &s) {
Index: sectionsd.cpp
===================================================================
RCS file: /cvs/tuxbox/apps/tuxbox/neutrino/daemons/sectionsd/sectionsd.cpp,v
retrieving revision 1.288
diff -u -r1.288 sectionsd.cpp
--- a/sectionsd.cpp	28 Feb 2009 13:57:50 -0000	1.288
+++ b/sectionsd.cpp	6 Mar 2009 08:31:18 -0000
@@ -667,8 +667,10 @@
 			writeLockEvents();
 			if (e->runningStatus() > 2) { // paused or currently running
 				if (!myCurrentEvent || (myCurrentEvent && (*myCurrentEvent).uniqueKey() != e->uniqueKey())) {
-					if (myCurrentEvent)
+					if (myCurrentEvent){
 						delete myCurrentEvent;
+						myCurrentEvent=NULL;
+					 }
 					myCurrentEvent = new SIevent(evt);
 					writeLockMessaging();
 					messaging_got_CN |= 0x01;
@@ -689,8 +691,10 @@
 			} else {
 				if ((!myNextEvent    || (myNextEvent    && (*myNextEvent).uniqueKey()    != e->uniqueKey() && (*myNextEvent).times.begin()->startzeit < e->times.begin()->startzeit)) &&
 				    (!myCurrentEvent || (myCurrentEvent && (*myCurrentEvent).uniqueKey() != e->uniqueKey()))) {
-					if (myNextEvent)
+					if (myNextEvent){
 						delete myNextEvent;
+						myNextEvent=NULL;
+					}
 					myNextEvent = new SIevent(evt);
 					writeLockMessaging();
 					messaging_got_CN |= 0x02;

Beitrag von **seife** » Freitag 6. März 2009, 10:25

re_Look hat geschrieben:Ich hab noch ein Patch.
Wenn jemand es testen will.
Bitte Speicher verbrauch in Auge behalten.

Die initialisierungen kann man machen (aber bitte im selben Codingstyle wie die Umgebung

), den Rest kommentier ich inline...

Code: Alles auswählen

Index: SIevents.cpp
===================================================================
RCS file: /cvs/tuxbox/apps/tuxbox/neutrino/daemons/sectionsd/SIevents.cpp,v
retrieving revision 1.35
diff -u -r1.35 SIevents.cpp
--- a/SIevents.cpp	16 Aug 2008 19:23:18 -0000	1.35
+++ b/SIevents.cpp	6 Mar 2009 08:31:14 -0000
@@ -75,6 +75,7 @@
 	transport_stream_id = _transport_stream_id;
 	service_id          = _service_id;
 	eventID		    = _event_id;
+	running=0;
 /*	contentClassification = "";
 	userClassification = "";
 	itemDescription = "";
Index: SIevents.hpp
===================================================================
RCS file: /cvs/tuxbox/apps/tuxbox/neutrino/daemons/sectionsd/SIevents.hpp,v
retrieving revision 1.29
diff -u -r1.29 SIevents.hpp
--- a/SIevents.hpp	16 Aug 2008 19:23:18 -0000	1.29
+++ b/SIevents.hpp	6 Mar 2009 08:31:15 -0000
@@ -341,6 +341,7 @@
 		original_network_id = 0;
 		transport_stream_id = 0;
 		eventID    = 0;
+		running=0;
 //      dauer=0;
 //      startzeit=0;
 	}
Index: SIservices.hpp
===================================================================
RCS file: /cvs/tuxbox/apps/tuxbox/neutrino/daemons/sectionsd/SIservices.hpp,v
retrieving revision 1.15
diff -u -r1.15 SIservices.hpp
--- a/SIservices.hpp	24 Feb 2009 19:09:10 -0000	1.15
+++ b/SIservices.hpp	6 Mar 2009 08:31:15 -0000
@@ -139,6 +139,7 @@
 		transport_stream_id = _transport_stream_id;
 		serviceTyp=0;
 		memset(&flags, 0, sizeof(flags));
+		is_actual = false;
 	}
 	// Std-Copy
 	SIservice(const SIservice &s) {
Index: sectionsd.cpp
===================================================================
RCS file: /cvs/tuxbox/apps/tuxbox/neutrino/daemons/sectionsd/sectionsd.cpp,v
retrieving revision 1.288
diff -u -r1.288 sectionsd.cpp
--- a/sectionsd.cpp	28 Feb 2009 13:57:50 -0000	1.288
+++ b/sectionsd.cpp	6 Mar 2009 08:31:18 -0000
@@ -667,8 +667,10 @@
 			writeLockEvents();
 			if (e->runningStatus() > 2) { // paused or currently running
 				if (!myCurrentEvent || (myCurrentEvent && (*myCurrentEvent).uniqueKey() != e->uniqueKey())) {
-					if (myCurrentEvent)
+					if (myCurrentEvent){
 						delete myCurrentEvent;
+						myCurrentEvent=NULL;
+					 }

Unnötig, weil:

Code: Alles auswählen

 					myCurrentEvent = new SIevent(evt);
 					writeLockMessaging();
 					messaging_got_CN |= 0x01;
@@ -689,8 +691,10 @@
 			} else {
 				if ((!myNextEvent    || (myNextEvent    && (*myNextEvent).uniqueKey()    != e->uniqueKey() && (*myNextEvent).times.begin()->startzeit < e->times.begin()->startzeit)) &&
 				    (!myCurrentEvent || (myCurrentEvent && (*myCurrentEvent).uniqueKey() != e->uniqueKey()))) {
-					if (myNextEvent)
+					if (myNextEvent){
 						delete myNextEvent;
+						myNextEvent=NULL;
+					}

dasselbe

Code: Alles auswählen

 					myNextEvent = new SIevent(evt);
 					writeLockMessaging();
 					messaging_got_CN |= 0x02;

Beitrag von **re_Look** » Freitag 6. März 2009, 11:17

Einverstanden "Chef".

Beitrag von **seife** » Freitag 6. März 2009, 11:39

Und nur um keine Missverständnisse aufkommen zu lassen: ich finde es gut, dass du den Code systematisch durchschaust und bin dir dankbar dafür.

Beitrag von **PauleFoul** » Freitag 6. März 2009, 13:24

seife hat geschrieben:Und nur um keine Missverständnisse aufkommen zu lassen: ich finde es gut, dass du den Code systematisch durchschaust und bin dir dankbar dafür.

DITO!!

Beitrag von **PT-1** » Freitag 6. März 2009, 15:48

Klasse Sache !

Beitrag von **mb405** » Freitag 6. März 2009, 16:50

ich habe seit langen mal mein cvs updatet, und image gebaut.
mit dem neuen sectionsd löscht mir der automatische hintergrundscan immer die kanäle

der 1.286 geht wunderbar.

kann das wer verifizieren ?

Beitrag von **JimPoison** » Freitag 6. März 2009, 18:48

ja, das problem hab ich auch neuerdings. dachte schon an einen fehler meiner box und wollte neuflashen, aber das kann ich mir dann wohl sparen.
muß noch dazu sagen, daß bei mir immer kanäle des bouquets gelöscht werden, von dem ich gerade einen sender schaue. besonders der aktuell geschaute kanal verschwindet einfach, die box schaltet auf kanal 1 (bei mir ard) und in den bouquetlisten fehlen dann einige kanäle des vorher benutzten bouquets.
kurz vorm verschwinden eines senders wurde öfter die meldung angezeigt, daß der hintergrundcan neue kanäle gefunden hat, aber das neu angelegte bouquet "neue sender" war immer leer

seitdem ich vor ein paar stunden mal testweise den automatischen hintergrundscan ausgeschaltet habe, gibts keine probleme mehr

Beitrag von **mb405** » Freitag 6. März 2009, 19:17

klar denn du hast in /tmp ne currentservices.xml datei wo drin steht was er macht

da steht bei mir mit dem neuesten sectionsd dimmer "del" für löschen. ansonsten steht da.add und replace.

Code: Alles auswählen

...
       <sat name="Astra 19.2E" position="0192" diseqc="0">
		<transponder id="0441" onid="0001" frequency="12187500" inversion="2" symbol_rate="27500000" fec_inner="3" polarization="0">
			<channel action="add" service_id="2ee6" name="RTL FS" service_type="01"/>
			<channel action="replace" service_id="2f30" name="Channel 21" service_type="01"/>
			<channel action="replace" service_id="2f3f" name="Channel 21" service_type="01"/>
		</transponder>
	</sat>
...

Beitrag von **seife** » Freitag 6. März 2009, 23:10

reverte mal das: http://cvs.tuxbox-cvs.sourceforge.net/l ... 00213.html
Ich denke, dass da am ehesten der Wurm drinsteckt.

Beitrag von **seife** » Samstag 7. März 2009, 00:28

Ich habe es (hoffentlich) gefixt.
Allerdings hat es mir die bouquets nicht zerschossen, drum hatte ich das nicht gemerkt (liegt evtl. daran, dass ich "Bouquets nicht ändern" in meiner Kanalsuche eingestellt habe).

Beitrag von **mb405** » Samstag 7. März 2009, 10:08

nein die bouquets bleiben.
er sorgt durch die del einträge in der currentservices.xml für ein löschen der kanäle. in den bouquets sind se noch drin, nur in den services dann nimmer.

Tuxbox-Board

[BUG] Sectionsd Bug ?

[BUG] Sectionsd Bug ?

Re: [BUG] Sectionsd Bug ?

Re: [BUG] Sectionsd Bug ?

Re: [BUG] Sectionsd Bug ?

Re: [BUG] Sectionsd Bug ?

Re: [BUG] Sectionsd Bug ?

Re: [BUG] Sectionsd Bug ?

Re: [BUG] Sectionsd Bug ?

Re: [BUG] Sectionsd Bug ?

Re: [BUG] Sectionsd Bug ?

Re: [BUG] Sectionsd Bug ?

Re: [BUG] Sectionsd Bug ?

Re: [BUG] Sectionsd Bug ?

Re: [BUG] Sectionsd Bug ?

Re: [BUG] Sectionsd Bug ?

Re: [BUG] Sectionsd Bug ?

Re: [BUG] Sectionsd Bug ?

Re: [BUG] Sectionsd Bug ?

Re: [BUG] Sectionsd Bug ?

Re: [BUG] Sectionsd Bug ?

Re: [BUG] Sectionsd Bug ?

Re: [BUG] Sectionsd Bug ?

Re: [BUG] Sectionsd Bug ?

Re: [BUG] Sectionsd Bug ?

Re: [BUG] Sectionsd Bug ?