whytofly
/
qt6-bb10
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

QFileDevice: clear cached permissions on setPermission() 

In theory, if we succeed, the permissions should be what we set, but
let's not make that assumption. And if we failed, it might be because
the file disappeared or something else, so re-stat()ing the file is a
good idea.

Pick-to: 6.4
Fixes: QTBUG-7211
Change-Id: If5d5ef6220874ae8858efffd171255506b7bbee0
Reviewed-by: Mårten Nordheim <marten.nordheim@qt.io>
bb10
Thiago Macieira 2022-09-06 10:23:36 -07:00
parent 68f641095c
commit 4b997d1851
3 changed files with 14 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/corelib/io/qfsfileengine_unix.cpp
View File

@ -489,6 +489,10 @@ bool QFSFileEngine::setPermissions(uint perms)
Q_D(QFSFileEngine);
QSystemError error;
bool ok;
// clear cached state (if any)
d->metaData.clearFlags(QFileSystemMetaData::Permissions);
if (d->fd != -1)
ok = QFileSystemEngine::setPermissions(d->fd, QFile::Permissions(perms), error);
else

4
src/corelib/io/qfsfileengine_win.cpp
View File

@ -655,6 +655,10 @@ bool QFSFileEngine::setPermissions(uint perms)
{
Q_D(QFSFileEngine);
QSystemError error;
// clear cached state (if any)
d->metaData.clearFlags(QFileSystemMetaData::Permissions);
bool ret = QFileSystemEngine::setPermissions(d->fileEntry, QFile::Permissions(perms), error);
if (!ret)
setError(QFile::PermissionsError, error.toString());

6
tests/auto/corelib/io/qfile/tst_qfile.cpp
View File

@ -1423,6 +1423,9 @@ void tst_QFile::permissionsNtfs()
void tst_QFile::setPermissions()
{
#ifdef Q_OS_QNX
QSKIP("This test doesn't pass on QNX and no one has cared to investigate.");
#endif
if ( QFile::exists( "createme.txt" ) )
QFile::remove( "createme.txt" );
QVERIFY( !QFile::exists( "createme.txt" ) );
@ -1433,9 +1436,12 @@ void tst_QFile::setPermissions()
f.close();
QFile::Permissions perms(QFile::WriteUser | QFile::ReadUser);
QVERIFY(f.setPermissions(QFile::ReadUser));
QVERIFY((f.permissions() & perms) == QFile::ReadUser);
QVERIFY(f.setPermissions(perms));
QVERIFY((f.permissions() & perms) == perms);
// we should end the test with the file in writeable state
}
void tst_QFile::copy()