Segfault mit Streamdev-Server-0.5.1+git20111021-0yavdr0~natty

Pete248 · 21. Oktober 2011

Eben lief meine yaVDR 0.4 Server/Client Kombi noch.

Dann schnell ein apt-get upgrade gemacht und das neue Streamdev-Server Plugin produziert eine VDR Restartschleife, wenn der Client ein Live-Streaming abruft.
Im Syslog steht: streamdev serve[23568]: segfault at 20 ip 00007fc644e9b436 sp 00007fc630ff8210 error 4 in libc-2.13.so[7fc644e54000+18a000]

Händisch ein älteres Streamdev-Server Plugin in den Plugin Ordner kopiert und der Spuk ist vorbei. Uff, Abend gerettet.

Ergo, es ist besser nicht gleich alle Clients upzudaten.

Am Streamdev-Plugin müsst ihr nochmal feilen, oder eine ältere Version ins Repository stellen, sonst gibt es lange Gesichter.

Pete

hotzenplotz5 · 21. Oktober 2011

jop hast recht!

Code

Program terminated with signal 11, Segmentation fault.
#0  0x00007f6f22847436 in vfprintf () from /lib/x86_64-linux-gnu/libc.so.6
#0  0x00007f6f22847436 in vfprintf () from /lib/x86_64-linux-gnu/libc.so.6
#1  0x00007f6f228fdbb8 in __vasprintf_chk () from /lib/x86_64-linux-gnu/libc.so.6
#2  0x00007f6f194994ea in cServerConnection::Respond(char const*, ...) () from /usr/lib/vdr/plugins/libvdr-streamdev-server.so.1.7.21
#3  0x00007f6f1949c1df in cConnectionVTP::Respond(int, char const*, ...) () from /usr/lib/vdr/plugins/libvdr-streamdev-server.so.1.7.21
#4  0x00007f6f194987cc in cStreamdevServer::Action() () from /usr/lib/vdr/plugins/libvdr-streamdev-server.so.1.7.21
#5  0x0000000000521113 in cThread::StartThread (Thread=0x7f6f12e6eee0) at thread.c:257
#6  0x00007f6f24031d8c in start_thread () from /lib/x86_64-linux-gnu/libpthread.so.0
#7  0x00007f6f228e604d in clone () from /lib/x86_64-linux-gnu/libc.so.6
#8  0x0000000000000000 in ?? ()

Alles anzeigen

schmirl help !

hotzenplotz5 · 22. Oktober 2011

so hab jetzt schon mal mehr :

Code

Core was generated by `/usr/bin/vdr-dbg --lirc=/var/run/lirc/lircd -v /srv/vdr/video.00 -c /var/lib/vd'.
Program terminated with signal 11, Segmentation fault.
#0  0x00007f0e63ba1436 in vfprintf () from /lib/x86_64-linux-gnu/libc.so.6
#0  0x00007f0e63ba1436 in vfprintf () from /lib/x86_64-linux-gnu/libc.so.6
#1  0x00007f0e63bc5b96 in vasprintf () from /lib/x86_64-linux-gnu/libc.so.6
#2  0x00007f0e5be6160f in cServerConnection::Respond (this=0x1f57a90, Message=0x7f0e5be9b8d3 "%03d%c%s") at connection.c:211
#3  0x00007f0e5be6ad17 in cConnectionVTP::Respond (this=0x1f57a90, Code=220, Message=0x7f0e5be9aca8 "Welcome to Video Disk Recorder (VTP)") at connectionVTP.c:1812
#4  0x00007f0e5be6624f in cConnectionVTP::Welcome (this=0x1f57a90) at connectionVTP.c:783
#5  0x00007f0e5be5f82e in cStreamdevServer::Action (this=0x7f0e4a5b6420) at server.c:137
#6  0x0000000000521113 in cThread::StartThread (Thread=0x7f0e4a5b6420) at thread.c:257
#7  0x00007f0e6538bd8c in start_thread () from /lib/x86_64-linux-gnu/libpthread.so.0
#8  0x00007f0e63c4004d in clone () from /lib/x86_64-linux-gnu/libc.so.6
#9  0x0000000000000000 in ?? ()

Alles anzeigen

hotzenplotz5 · 22. Oktober 2011

connection.c zeile 211:

Code

length = vasprintf(&buffer, Message, ap);

der block sieht so aus :

Code

bool cServerConnection::Respond(const char *Message, ...) 
{
	char *buffer;
	int length;
	va_list ap;
	va_start(ap, Message);
	length = vasprintf(&buffer, Message, ap);
	va_end(ap);


	if (length < 0) {
		esyslog("ERROR: streamdev: buffer allocation failed (%s) for %s:%d",
				m_Protocol, RemoteIp().c_str(), RemotePort());
		return false;
	}


	if (m_WriteBytes + length + 2 > sizeof(m_WriteBuffer)) {
		esyslog("ERROR: streamdev: output buffer overflow (%s) for %s:%d", 
		        m_Protocol, RemoteIp().c_str(), RemotePort());
		free(buffer);
		return false;bool cServerConnection::Respond(const char *Message, ...) 
{
	char *buffer;
	int length;
	va_list ap;
	va_start(ap, Message);
	length = vasprintf(&buffer, Message, ap);
	va_end(ap);


	if (length < 0) {
		esyslog("ERROR: streamdev: buffer allocation failed (%s) for %s:%d",
				m_Protocol, RemoteIp().c_str(), RemotePort());
		return false;
	}


	if (m_WriteBytes + length + 2 > sizeof(m_WriteBuffer)) {
		esyslog("ERROR: streamdev: output buffer overflow (%s) for %s:%d", 
		        m_Protocol, RemoteIp().c_str(), RemotePort());
		free(buffer);
		return false;
	}
	Dprintf("OUT: |%s|\n", buffer);
	memcpy(m_WriteBuffer + m_WriteBytes, buffer, length);
	free(buffer);


	m_WriteBytes += length;
	m_WriteBuffer[m_WriteBytes++] = '\015';
	m_WriteBuffer[m_WriteBytes++] = '\012';
	return true;
}


	}
	Dprintf("OUT: |%s|\n", buffer);
	memcpy(m_WriteBuffer + m_WriteBytes, buffer, length);
	free(buffer);


	m_WriteBytes += length;
	m_WriteBuffer[m_WriteBytes++] = '\015';
	m_WriteBuffer[m_WriteBytes++] = '\012';
	return true;
}

Alles anzeigen

connectionVTP.c zeile 1812:

Code

Code < 0 ? '-' : ' ', *str);

und der "ganze" block:

Code

bool cConnectionVTP::Respond(int Code, const char *Message, ...)
{
	va_list ap;
	va_start(ap, Message);
	cString str = cString::sprintf(Message, ap);
	va_end(ap);


	if (Code >= 0 && m_LastCommand != NULL) {
		free(m_LastCommand);
		m_LastCommand = NULL;
	}


	return cServerConnection::Respond("%03d%c%s", Code >= 0, 
				Code < 0 ? -Code : Code,
				Code < 0 ? '-' : ' ', *str);
}

Alles anzeigen

hotzenplotz5 · 22. Oktober 2011

vielleicht passt das vom letzten commit dazu :

http://projects.vdr-developer.…49d7a2253ac937130b6e11d8d

Code

@@ -205,12 +202,12 @@ bool cServerConnection::Write(void)
return true;
}
-bool cServerConnection::Respond(const char *Message, bool Last, ...)
+bool cServerConnection::Respond(const char *Message, ...)
{
char *buffer;
int length;
va_list ap;
- va_start(ap, Last);
+ va_start(ap, Message);
length = vasprintf(&buffer, Message, ap);
va_end(ap);

Alles anzeigen

mini73 · 22. Oktober 2011

Moin!

Code

return cServerConnection::Respond("%03d%c%s", Code >= 0, Code < 0 ? -Code : Code, Code < 0 ? '-' : ' ', *str);

Drei Platzhalter und vier Argumente - das passt nicht zusammen. Das "Code >= 0" ist wohl noch ein Überbleibsel für "Last", den es nicht mehr gibt. Vermutlich entfernen... oder noch einen Platzhalter einbauen.

Code

return cServerConnection::Respond("%03d%c%s", Code < 0 ? -Code : Code, Code < 0 ? '-' : ' ', *str);

Lars.

hotzenplotz5 · 22. Oktober 2011

danke lars ! damit läuft es wieder :

Diff

--- vdr-plugin-streamdev-0.5.1+git20111022.orig/server/connectionVTP.c
+++ vdr-plugin-streamdev-0.5.1+git20111022/server/connectionVTP.c
@@ -1807,7 +1807,7 @@ bool cConnectionVTP::Respond(int Code, c
 		m_LastCommand = NULL;
 	}
 
-	return cServerConnection::Respond("%03d%c%s", Code >= 0, 
+	return cServerConnection::Respond("%03d%c%s", 
 				Code < 0 ? -Code : Code,
 				Code < 0 ? '-' : ' ', *str);
 }

Alles anzeigen

Pete248 · 23. Oktober 2011

Das heißt, man darf jetzt wieder apt-get upgraden ?

Pete

hotzenplotz5 · 23. Oktober 2011

ich hab im ppa die letzte version die nicht crashte.
also nicht die neueste mit patch.
die läuft bisher nur bei mir.
ich wollte auf schmirl warten ob das so richtig ist.
hier läuft es, aber das soll ja nix heissen.

Segfault mit Streamdev-Server-0.5.1+git20111021-0yavdr0~natty

Jetzt mitmachen!

Teilen