Merge pull request 'Tor: properly overwrite old binary' (#334) from tobtoht/feather:docker_tor_openssl into master

Reviewed-on: https://git.wownero.com/feather/feather/pulls/334
This commit is contained in:
tobtoht 2021-02-05 18:43:55 +00:00
commit c305bfd5a6
2 changed files with 10 additions and 2 deletions

View file

@ -211,8 +211,12 @@ bool Tor::unpackBins() {
TorVersion embeddedVersion = this->stringToVersion(QString(TOR_VERSION));
TorVersion filesystemVersion = this->getVersion(torPath);
qDebug() << QString("Tor versions: embedded %1, filesystem %2").arg(embeddedVersion.toString(), filesystemVersion.toString());
if (embeddedVersion > filesystemVersion) {
QFile::remove(torPath);
if (TorVersion::isValid(filesystemVersion) && (embeddedVersion > filesystemVersion)) {
qInfo() << "Embedded version is newer, overwriting.";
QFile::setPermissions(torPath, QFile::ReadOther | QFile::WriteOther);
if (!QFile::remove(torPath)) {
qWarning() << "Unable to remove old Tor binary";
};
}
qDebug() << "Writing Tor executable to " << this->torPath;

View file

@ -54,6 +54,10 @@ struct TorVersion
QString::number(patch), QString::number(release));
}
static bool isValid(const TorVersion &v) {
return v != TorVersion();
}
int major;
int minor;
int patch;