Index: head/devel/kf5-kconfig/Makefile =================================================================== --- head/devel/kf5-kconfig/Makefile (revision 508480) +++ head/devel/kf5-kconfig/Makefile (revision 508481) @@ -1,20 +1,20 @@ # $FreeBSD$ PORTNAME= kconfig DISTVERSION= ${KDE_FRAMEWORKS_VERSION} -PORTREVISION= 1 +PORTREVISION= 2 CATEGORIES= devel kde kde-frameworks MAINTAINER= kde@FreeBSD.org COMMENT= KF5 widgets for configuration dialogs # Sources say LGPL20+, KDE policy says LGPL21+, license file is LGPL21 LICENSE= LGPL21+ LICENSE_FILE= ${WRKSRC}/COPYING.LIB USES= cmake compiler:c++11-lib kde:5 qt:5 tar:xz USE_KDE= ecm USE_QT= buildtools_build concurrent core dbus gui linguisttools \ qmake_build testlib xml .include Index: head/devel/kf5-kconfig/files/patch-git-5d3e71b1d2ecd2cb2f910036e614ffdfc895aa22 =================================================================== --- head/devel/kf5-kconfig/files/patch-git-5d3e71b1d2ecd2cb2f910036e614ffdfc895aa22 (nonexistent) +++ head/devel/kf5-kconfig/files/patch-git-5d3e71b1d2ecd2cb2f910036e614ffdfc895aa22 (revision 508481) @@ -0,0 +1,139 @@ +commit 5d3e71b1d2ecd2cb2f910036e614ffdfc895aa22 +Author: David Faure +Date: Wed Aug 7 09:35:36 2019 +0200 + + 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 +diff --git autotests/kconfigtest.cpp autotests/kconfigtest.cpp +index 410b5b8..9af3b46 100644 +--- autotests/kconfigtest.cpp ++++ autotests/kconfigtest.cpp +@@ -38,7 +38,7 @@ + #include + #endif + #ifndef Q_OS_WIN +-#include // gethostname ++#include // getuid + #endif + + KCONFIGGROUP_DECLARE_ENUM_QOBJECT(KConfigTest, Testing) +@@ -546,14 +546,8 @@ void KConfigTest::testPath() + QCOMPARE(group.readPathEntry("withBraces", QString()), QString("file://" + HOMEPATH)); + QVERIFY(group.hasKey("URL")); + QCOMPARE(group.readEntry("URL", QString()), QString("file://" + HOMEPATH)); +-#if !defined(Q_OS_WIN32) && !defined(Q_OS_MAC) +- // I don't know if this will work on windows +- // This test hangs on OS X + QVERIFY(group.hasKey("hostname")); +- char hostname[256]; +- QVERIFY(::gethostname(hostname, sizeof(hostname)) == 0); +- QCOMPARE(group.readEntry("hostname", QString()), QString::fromLatin1(hostname)); +-#endif ++ QCOMPARE(group.readEntry("hostname", QString()), QStringLiteral("(hostname)")); // the $ got removed because empty var name + QVERIFY(group.hasKey("noeol")); + QCOMPARE(group.readEntry("noeol", QString()), QString("foo")); + +diff --git docs/options.md docs/options.md +index c634c00..4a6e9bc 100644 +--- docs/options.md ++++ docs/options.md +@@ -67,18 +67,15 @@ environment variables (and `XDG_CONFIG_HOME` in particular). + Shell Expansion + --------------- + +-If an entry is marked with `$e`, environment variables and shell commands will +-be expanded. ++If an entry is marked with `$e`, environment variables will be expanded. + + Name[$e]=$USER +- Host[$e]=$(hostname) + + When the "Name" entry is read `$USER` will be replaced with the value of the +-`$USER` environment variable, and `$(hostname)` will be replaced with the output +-of the `hostname` command. ++`$USER` environment variable. + +-Note that the application will replace `$USER` and `$(hostname)` with their +-respective expanded values after saving. To prevent this combine the `$e` option ++Note that the application will replace `$USER` with its ++expanded value after saving. To prevent this combine the `$e` option + with `$i` (immmutable) option. For example: + + Name[$ei]=$USER +diff --git src/core/kconfig.cpp src/core/kconfig.cpp +index e1b11ed..f6824ce 100644 +--- src/core/kconfig.cpp ++++ src/core/kconfig.cpp +@@ -28,19 +28,6 @@ + #include + #include + +-#ifdef _MSC_VER +-static inline FILE *popen(const char *cmd, const char *mode) +-{ +- return _popen(cmd, mode); +-} +-static inline int pclose(FILE *stream) +-{ +- return _pclose(stream); +-} +-#else +-#include +-#endif +- + #include "kconfigbackend_p.h" + #include "kconfiggroup.h" + +@@ -183,29 +170,7 @@ QString KConfigPrivate::expandString(const QString &value) + int nDollarPos = aValue.indexOf(QLatin1Char('$')); + while (nDollarPos != -1 && nDollarPos + 1 < aValue.length()) { + // there is at least one $ +- if (aValue[nDollarPos + 1] == QLatin1Char('(')) { +- int nEndPos = nDollarPos + 1; +- // the next character is not $ +- while ((nEndPos <= aValue.length()) && (aValue[nEndPos] != QLatin1Char(')'))) { +- nEndPos++; +- } +- nEndPos++; +- QString cmd = aValue.mid(nDollarPos + 2, nEndPos - nDollarPos - 3); +- +- QString result; +- +-// FIXME: wince does not have pipes +-#ifndef _WIN32_WCE +- FILE *fs = popen(QFile::encodeName(cmd).data(), "r"); +- if (fs) { +- QTextStream ts(fs, QIODevice::ReadOnly); +- result = ts.readAll().trimmed(); +- pclose(fs); +- } +-#endif +- aValue.replace(nDollarPos, nEndPos - nDollarPos, result); +- nDollarPos += result.length(); +- } else if (aValue[nDollarPos + 1] != QLatin1Char('$')) { ++ if (aValue[nDollarPos + 1] != QLatin1Char('$')) { + int nEndPos = nDollarPos + 1; + // the next character is not $ + QStringRef aVarName; Property changes on: head/devel/kf5-kconfig/files/patch-git-5d3e71b1d2ecd2cb2f910036e614ffdfc895aa22 ___________________________________________________________________ Added: fbsd:nokeywords ## -0,0 +1 ## +yes \ No newline at end of property Added: svn:eol-style ## -0,0 +1 ## +native \ No newline at end of property Added: svn:mime-type ## -0,0 +1 ## +text/plain \ No newline at end of property