fixed upnp bug that could cause stack overflow
This commit is contained in:
parent
7fff6ace1b
commit
25dd5c86b9
|
@ -61,6 +61,7 @@ release 0.14.2
|
||||||
from resume data properly
|
from resume data properly
|
||||||
* removed locale dependency in xml parser (caused asserts on windows)
|
* removed locale dependency in xml parser (caused asserts on windows)
|
||||||
* fixed bug when talking to https 1.0 servers
|
* fixed bug when talking to https 1.0 servers
|
||||||
|
* fixed UPnP bug that could cause stack overflow
|
||||||
|
|
||||||
release 0.14.1
|
release 0.14.1
|
||||||
|
|
||||||
|
|
|
@ -637,6 +637,7 @@ void upnp::update_map(rootdevice& d, int i)
|
||||||
std::stringstream msg;
|
std::stringstream msg;
|
||||||
msg << "mapping " << i << " does not need updating, skipping";
|
msg << "mapping " << i << " does not need updating, skipping";
|
||||||
log(msg.str());
|
log(msg.str());
|
||||||
|
m.action = mapping_t::action_none;
|
||||||
next(d, i);
|
next(d, i);
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
|
@ -651,6 +652,7 @@ void upnp::update_map(rootdevice& d, int i)
|
||||||
{
|
{
|
||||||
if (m.failcount > 5)
|
if (m.failcount > 5)
|
||||||
{
|
{
|
||||||
|
m.action = mapping_t::action_none;
|
||||||
// giving up
|
// giving up
|
||||||
next(d, i);
|
next(d, i);
|
||||||
return;
|
return;
|
||||||
|
|
Loading…
Reference in New Issue