diff --git a/src/libcalamares/utils/Permissions.h b/src/libcalamares/utils/Permissions.h index 05b3c19c6..9e0357b57 100644 --- a/src/libcalamares/utils/Permissions.h +++ b/src/libcalamares/utils/Permissions.h @@ -61,6 +61,8 @@ public: /** @brief Sets the file-access @p mode of @p path * * Pass a path that is relative (or absolute) in the **host** system. + * + * @return @c true on success */ static bool apply( const QString& path, int mode ); /** @brief Do both chmod and chown on @p path @@ -72,6 +74,8 @@ public: * * For this call, the names are interpreted in the **host** system. * Pass a path that is relative (or absolute) in the **host** system. + * + * @return @c true on success of **both** operations */ static bool apply( const QString& path, const Permissions& p ); /// Convenience method for apply(const QString&, const Permissions& ) diff --git a/src/modules/users/CreateUserJob.cpp b/src/modules/users/CreateUserJob.cpp index a6812dd53..6ccd678cc 100644 --- a/src/modules/users/CreateUserJob.cpp +++ b/src/modules/users/CreateUserJob.cpp @@ -177,7 +177,7 @@ CreateUserJob::exec() if ( fileResult ) { - if ( CalamaresUtils::Permissions::apply( fileResult.path(), 0440 ) ) + if ( !CalamaresUtils::Permissions::apply( fileResult.path(), 0440 ) ) { return Calamares::JobResult::error( tr( "Cannot chmod sudoers file." ) ); }