Differential D27388

Fix being unable to set the right permissions for kde_settings.conf
ClosedPublic
Actions

Authored by filipf on Feb 13 2020, 11:49 PM.

Details

Reviewers
davidedmundson
Commits
R123:cc33ec575c5e: Fix being unable to set the right permissions for kde_settings.conf
Summary

If we are creating a config file and its directory doesn't exist, we fail to set the right permissions for the file.

This affects kde_settings.conf and results in the KCM not being able to read stored settings.

BUG: 414396
CC-BUG: 342722
FIXED-IN: 5.18.1

Test Plan

rm-rf sddm.conf.d
save settings in the kcm in various ways
kde_settings.conf is now world readable agian

Diff Detail

Repository
R123 SDDM Configuration Panel (KCM)
Branch
fix-config-file-permissions (branched from master)
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 22457
Build 22475: arc lint + arc unit
filipf created this revision.Feb 13 2020, 11:49 PM
Restricted Application added a project: Plasma. · View Herald TranscriptFeb 13 2020, 11:49 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
filipf requested review of this revision.Feb 13 2020, 11:49 PM
filipf updated this revision to Diff 75653.Feb 14 2020, 12:02 AM

Fix the permissions for people who have already had the file created with the wrong permissions

davidedmundson accepted this revision.Feb 16 2020, 7:05 AM
This revision is now accepted and ready to land.Feb 16 2020, 7:05 AM
Closed by commit R123:cc33ec575c5e: Fix being unable to set the right permissions for kde_settings.conf (authored by filipf). · Explain WhyFeb 16 2020, 11:55 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Msddmauthhelper.cpp (11 lines)

Diff 75653

View Options

sddmauthhelper.cpp

Loading...