ETM-Schurig/powersync-core
Archived
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

fixed memory leek from network reply and set power bug

Browse Source
This commit is contained in:
Simon Stürz 2014-08-08 17:58:30 +02:00 committed by Michael Zanetti
parent 50e1ddfc5f
commit 22d9bd312f
1 changed files with 3 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
plugins/deviceplugins/wemo/wemoswitch.cpp
View File

@ -155,6 +155,7 @@ void WemoSwitch::replyFinished(QNetworkReply *reply)
if(data.contains("<BinaryState>1</BinaryState>")){ if(data.contains("<BinaryState>1</BinaryState>")){
m_powerState = true; m_powerState = true;
} }
m_refrashReplay->deleteLater();
} }
// if this is the answerer to a "set power" request // if this is the answerer to a "set power" request
if(reply == m_setPowerReplay){ if(reply == m_setPowerReplay){
@ -165,6 +166,7 @@ void WemoSwitch::replyFinished(QNetworkReply *reply)
emit setPowerFinished(false,m_actionId); emit setPowerFinished(false,m_actionId);
} }
refresh(); refresh();
m_setPowerReplay->deleteLater();
} }
emit stateChanged(); emit stateChanged();
@ -189,6 +191,7 @@ void WemoSwitch::setPower(const bool &power, const ActionId &actionId)
if(m_powerState == power){ if(m_powerState == power){
emit setPowerFinished(true,actionId); emit setPowerFinished(true,actionId);
return;
} }
QByteArray setPowerMessage("<?xml version=\"1.0\" encoding=\"utf-8\"?><s:Envelope xmlns:s=\"http://schemas.xmlsoap.org/soap/envelope/\" s:encodingStyle=\"http://schemas.xmlsoap.org/soap/encoding/\"><s:Body><u:SetBinaryState xmlns:u=\"urn:Belkin:service:basicevent:1\"><BinaryState>" + QByteArray::number((int)power) + "</BinaryState></u:SetBinaryState></s:Body></s:Envelope>"); QByteArray setPowerMessage("<?xml version=\"1.0\" encoding=\"utf-8\"?><s:Envelope xmlns:s=\"http://schemas.xmlsoap.org/soap/envelope/\" s:encodingStyle=\"http://schemas.xmlsoap.org/soap/encoding/\"><s:Body><u:SetBinaryState xmlns:u=\"urn:Belkin:service:basicevent:1\"><BinaryState>" + QByteArray::number((int)power) + "</BinaryState></u:SetBinaryState></s:Body></s:Envelope>");