ONIMAI-SMP/ShatteredPrism
3
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 1 Wiki Activity

Merge pull request #2655 from Ryex/fix/json-parsing-validator-buffer-reset

Browse Source 
fix(net/validators): ensure buffers are cleared and reset;
This commit is contained in:
Tayou 2024-07-18 09:11:29 +02:00 committed by GitHub
commit dcc58c8b75
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
2 changed files with 15 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
launcher/meta/BaseEntity.cpp
View File

@ -30,13 +30,21 @@ class ParsingValidator : public Net::Validator {
virtual ~ParsingValidator() {}; virtual ~ParsingValidator() {};
public: /* methods */ public: /* methods */
bool init(QNetworkRequest&) override { return true; } bool init(QNetworkRequest&) override
{
m_data.clear();
return true;
}
bool write(QByteArray& data) override bool write(QByteArray& data) override
{ {
this->m_data.append(data); this->m_data.append(data);
return true; return true;
} }
bool abort() override { return true; } bool abort() override
{
m_data.clear();
return true;
}
bool validate(QNetworkReply&) override bool validate(QNetworkReply&) override
{ {
auto fname = m_entity->localFilename(); auto fname = m_entity->localFilename();

6
launcher/net/ChecksumValidator.h
View File

@ -60,7 +60,11 @@ class ChecksumValidator : public Validator {
return true; return true;
} }
auto abort() -> bool override { return true; } auto abort() -> bool override
{
m_checksum.reset();
return true;
}
auto validate(QNetworkReply&) -> bool override auto validate(QNetworkReply&) -> bool override
{ {