generic-library/libupnp
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Issue regarding the GENA notifications. A string termination indicator was added

Browse Source 
at the end of the notification ("\r\n") in notify_send_and_recv() in
upnp/src/gena/gena_device.c.

Patch by Fabrice Fontaine.
This commit is contained in:
Marcelo Roberto Jimenez 2010-09-03 21:49:49 -03:00
parent 842a6ce5c8
commit ebc941f265
2 changed files with 10 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
ChangeLog
View File

@ -2,6 +2,12 @@
Version 1.6.7 Version 1.6.7
******************************************************************************* *******************************************************************************
2010-08-21 Marcelo Jimenez <mroberto(at)users.sourceforge.net>
* Issue regarding the GENA notifications. A string termination indicator
was added at the end of the notification ("\r\n") in
notify_send_and_recv() in upnp/src/gena/gena_device.c.
Patch by Fabrice Fontaine.
2010-08-21 Marcelo Jimenez <mroberto(at)users.sourceforge.net> 2010-08-21 Marcelo Jimenez <mroberto(at)users.sourceforge.net>
* The last part of Ronan Menard's patch. * The last part of Ronan Menard's patch.

7
upnp/src/gena/gena_device.c
View File

@ -215,12 +215,13 @@ static UPNP_INLINE int notify_send_and_recv(
timeout = HTTP_DEFAULT_TIMEOUT; timeout = HTTP_DEFAULT_TIMEOUT;
// send msg (note +1 for propertyset; null-terminator is also sent) // send msg (note: end of notification will contain "\r\n" twice)
if( ( ret_code = http_SendMessage( &info, &timeout, if( ( ret_code = http_SendMessage( &info, &timeout,
"bb", "bbb",
start_msg.buf, start_msg.length, start_msg.buf, start_msg.length,
propertySet, propertySet,
strlen( propertySet ) + 1 ) ) != strlen( propertySet ),
"\r\n", strlen( "\r\n" ) ) ) !=
0 ) { 0 ) {
membuffer_destroy( &start_msg ); membuffer_destroy( &start_msg );
sock_destroy( &info, SD_BOTH ); sock_destroy( &info, SD_BOTH );