nmelone/GoldenCheetah
1
0
Fork 0
mirror of https://github.com/GoldenCheetah/GoldenCheetah.git synced 2026-04-13 12:42:20 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix crash when importing compressed zip files

Browse Source 
The bug was introduced by #4751 when QScopedPointer was replaced
by std::unique_ptr but release() was not called to replace take().
This commit is contained in:
Alejandro Martinez
2025-12-19 13:58:38 -03:00
parent 78fb79cfe1
commit be01a861b8
1 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
contrib/qzip/zip.cpp
View File

@@ -514,7 +514,7 @@ void ZipReaderPrivate::scanFiles()
int num_dir_entries = 0;
EndOfDirectory eod;
while (start_of_directory == -1) {
int pos = device->size() - sizeof(EndOfDirectory) - i;
size_t pos = device->size() - sizeof(EndOfDirectory) - i;
if (pos < 0 || i > 65535) {
qWarning() << "QZip: EndOfDirectory not found";
return;
@@ -755,7 +755,7 @@ ZipReader::ZipReader(const QString &archive, QIODevice::OpenMode mode)
status = FileError;
}
d = new ZipReaderPrivate(&(*f), /*ownDevice=*/true);
d = new ZipReaderPrivate(&(*f.release()), /*ownDevice=*/true);
d->status = status;
}
@@ -1039,7 +1039,7 @@ ZipWriter::ZipWriter(const QString &fileName, QIODevice::OpenMode mode)
status = ZipWriter::FileError;
}
d = new ZipWriterPrivate(&(*f), /*ownDevice=*/true);
d = new ZipWriterPrivate(&(*f.release()), /*ownDevice=*/true);
d->status = status;
}