D28510: [kprocessrunner] Escape hyphen characters in scope names

David Edmundson noreply at phabricator.kde.org
Thu Apr 2 11:12:26 BST 2020


davidedmundson created this revision.
Herald added a project: Frameworks.
Herald added a subscriber: kde-frameworks-devel.
davidedmundson requested review of this revision.

REVISION SUMMARY
  Desktop names can be from the set "ASCII letters, digits, dash (U+002D
  HYPHEN-MINUS) and underscore (U+005F LOW LINE). "
  
  Hyphen is a special character in systemd units indicating a logical tree
  level. We would process drop-ins incorrectly, Specification says they should be escaped.

TEST PLAN
  Ran elf-dissector
  systemdcg-ls showed
  org.kde.elf\x2ddissector-0dc652e69f5d4557873db10d03517b41.scope

REPOSITORY
  R241 KIO

BRANCH
  master

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

AFFECTED FILES
  src/gui/kprocessrunner.cpp

To: davidedmundson
Cc: kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20200402/06b5e990/attachment.html>


More information about the Kde-frameworks-devel mailing list