mifr/subsurface
1
0
Fork 0
mirror of https://github.com/subsurface/subsurface.git synced 2025-01-19 06:15:26 +00:00
Code Issues Projects Releases Packages Wiki Activity

export: use unique temporary file for divelogs.de upload

Browse source 
On multi-user systems with a shared directory for temporary files, using
a static file name can lead to permissions problems and subsequent
errors due to collisions. Use a random unique file name for each
generated file to avoid these problems.

Note: the temporary file generated from the divelogs.de upload is still
left behind after the upload finishes.

Signed-off-by: Richard Fuchs <dfx@dfx.at>
This commit is contained in:
Richard Fuchs 2021-08-02 11:55:53 -04:00 • committed by Dirk Hohndel
parent 55bc1938ad
commit cf78e4cb20
1 changed files with 13 additions and 11 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

24
core/uploadDiveLogsDE.cpp
View file

@ -2,6 +2,7 @@
#include "uploadDiveLogsDE.h"
#include <QDir>
#include <QDebug>
#include <QTemporaryFile>
#include <zip.h>
#include <errno.h>
#include "core/display.h"
@ -34,21 +35,22 @@ uploadDiveLogsDE::uploadDiveLogsDE():
}
static QString makeTempFileName()
{
QTemporaryFile tmpfile;
tmpfile.setFileTemplate(QDir::tempPath() + "/divelogsde-upload.XXXXXXXX.dld");
tmpfile.open();
QString filename(tmpfile.fileName());
tmpfile.close();
return filename;
}
void uploadDiveLogsDE::doUpload(bool selected, const QString &userid, const QString &password)
{
QString err;
/* generate a temporary filename and create/open that file with zip_open */
QString filename(QDir::tempPath() + "/divelogsde-upload.dld");
// delete file if it exist
QFile f(filename);
if (f.open(QIODevice::ReadOnly)) {
f.close();
f.remove();
}
QString filename = makeTempFileName();
// Make zip file, with all dives, in divelogs.de format
if (!prepareDives(filename, selected)) {