From 7ef089b09a79aa2e596d8ed7a5d04d74159d9882 Mon Sep 17 00:00:00 2001 From: Fabrice Fontaine Date: Tue, 6 Mar 2012 18:35:33 +0100 Subject: [PATCH] Removing two unused variables in ssdp_server.c Removing first TempPtr allocation in unique_service_name as well as one of the dbgStr allocation in AdvertizeAndReply as those values were not used. --- ChangeLog | 8 ++++++++ upnp/src/ssdp/ssdp_server.c | 3 +-- 2 files changed, 9 insertions(+), 2 deletions(-) diff --git a/ChangeLog b/ChangeLog index 4811644..7410884 100644 --- a/ChangeLog +++ b/ChangeLog @@ -2,6 +2,14 @@ Version 1.6.16 ******************************************************************************* +2012-03-06 Fabrice Fontaine + + Removing two unused variables in ssdp_server.c + + Removing first TempPtr allocation in unique_service_name as well as one + of the dbgStr allocation in AdvertizeAndReply as those values were not + used. + 2012-03-06 Fabrice Fontaine SF Bug Tracker id 3497714 - Buffer overflows diff --git a/upnp/src/ssdp/ssdp_server.c b/upnp/src/ssdp/ssdp_server.c index 7b494e3..90d5fbd 100644 --- a/upnp/src/ssdp/ssdp_server.c +++ b/upnp/src/ssdp/ssdp_server.c @@ -152,7 +152,6 @@ int AdvertiseAndReply(int AdFlag, UpnpDevice_Handle Hnd, continue; UpnpPrintf(UPNP_ALL, API, __FILE__, __LINE__, "Extracting UDN for %s\n", dbgStr); - dbgStr = ixmlNode_getNodeName(tmpNode); UpnpPrintf(UPNP_ALL, API, __FILE__, __LINE__, "Extracting device type\n"); tmpNode2 = ixmlNodeList_item(nodeList, 0); @@ -458,7 +457,7 @@ int unique_service_name(char *cmd, SsdpEvent *Evt) int CommandFound = 0; size_t n = 0; - if ((TempPtr = strstr(cmd, "uuid:schemas")) != NULL) { + if (strstr(cmd, "uuid:schemas") != NULL) { ptr1 = strstr(cmd, ":device"); if (ptr1 != NULL) ptr2 = strstr(ptr1 + 1, ":");