summaryrefslogtreecommitdiffstats
path: root/Help
diff options
context:
space:
mode:
authorJustin Clift <justin@postgresql.org>2016-07-15 13:18:37 (GMT)
committerBrad King <brad.king@kitware.com>2016-07-21 14:05:46 (GMT)
commit11768733d321df55b0efcb70b278c71b8e216cf7 (patch)
treef56cb0e1e1055e7d462a3508d284055b173ffa03 /Help
parente31084e65745f9dd422c6aff0a2ed4ada6918805 (diff)
downloadCMake-11768733d321df55b0efcb70b278c71b8e216cf7.zip
CMake-11768733d321df55b0efcb70b278c71b8e216cf7.tar.gz
CMake-11768733d321df55b0efcb70b278c71b8e216cf7.tar.bz2
NSIS: Quote uninstaller path when executing it in a shell
Protect our `$0` reference in the shell as `"$0"`. Otherwise it works with a space in the path only due to an insecure Windows feature. Prior to this fix, any installer using the option added by commit v2.8.9~234^2 (Added CPACK_NSIS_ENABLE_UNINSTALL_BEFORE_INSTALL, 2011-06-11) exposes a local privilege escalation vulnerability. Reported-by: Amir Szekely <kichik@gmail.com> Reported-by: Ug_0 Security
Diffstat (limited to 'Help')
-rw-r--r--Help/release/3.6.rst6
1 files changed, 6 insertions, 0 deletions
diff --git a/Help/release/3.6.rst b/Help/release/3.6.rst
index 771c9dd..144537d 100644
--- a/Help/release/3.6.rst
+++ b/Help/release/3.6.rst
@@ -308,3 +308,9 @@ Other Changes
preferred future use is upper cased component names in variables.
New variables that will be added to CPackRPM in later versions
will only support upper cased component variable format.
+
+* The CPack NSIS generator's configuration file template was fixed to
+ quote the path to the uninstaller tool used by the
+ :variable:`CPACK_NSIS_ENABLE_UNINSTALL_BEFORE_INSTALL` option.
+ This avoids depending on an insecure Windows feature to run an
+ uninstaller tool with a space in the path.