filipf created this revision.
filipf added a reviewer: davidedmundson.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
filipf requested review of this revision.

REVISION 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

REPOSITORY
  R123 SDDM Configuration Panel (KCM)

BRANCH
  fix-config-file-permissions (branched from master)

REVISION DETAIL
  https://phabricator.kde.org/D27388

AFFECTED FILES
  sddmauthhelper.cpp

To: filipf, davidedmundson
Cc: plasma-devel, ngraham, Orage, LeGast00n, The-Feren-OS-Dev, cblack, 
jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, alexeymin, himcesjf, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

Reply via email to