aboutsummaryrefslogtreecommitdiff
path: root/src/core/kconfigini.cpp
diff options
context:
space:
mode:
authorDavid Faure <faure@kde.org>2019-08-07 09:35:36 +0200
committerDavid Faure <faure@kde.org>2019-08-07 16:04:18 +0200
commit5d3e71b1d2ecd2cb2f910036e614ffdfc895aa22 (patch)
treeaa66599bc0c94f979ae78183cb9d0dd99d92bf45 /src/core/kconfigini.cpp
parent8e82d5accb52eda7bec784de447a9cb6161bbbb8 (diff)
downloadkconfig-5d3e71b1d2ecd2cb2f910036e614ffdfc895aa22.tar.gz
kconfig-5d3e71b1d2ecd2cb2f910036e614ffdfc895aa22.tar.bz2
Security: remove support for $(...) in config keys with [$e] marker.
Summary: It is very unclear at this point what a valid use case for this feature would possibly be. The old documentation only mentions $(hostname) as an example, which can be done with $HOSTNAME instead. Note that $(...) is still supported in Exec lines of desktop files, this does not require [$e] anyway (and actually works better without it, otherwise the $ signs need to be doubled to obey kconfig $e escaping rules...). Test Plan: ctest passes; various testcases with $(...) in desktop files, directory files, and config files, no longer execute commands. Reviewers: mdawson, aacid, broulik, davidedmundson, kossebau, apol, sitter, security-team Reviewed By: mdawson, davidedmundson Subscribers: ZaWertun, rikmills, fvogt, ngraham, kde-frameworks-devel Tags: #frameworks Differential Revision: https://phabricator.kde.org/D22979
Diffstat (limited to 'src/core/kconfigini.cpp')
0 files changed, 0 insertions, 0 deletions