uh oh
authorJoey Hess <joey@gnu.kitenet.net>
Sat, 22 Aug 2009 18:29:18 +0000 (14:29 -0400)
committerJoey Hess <joey@gnu.kitenet.net>
Sat, 22 Aug 2009 18:29:18 +0000 (14:29 -0400)
doc/plugins/contrib/cvs/discussion.mdwn

index 65b6befd17eadf97127d54d7fe2b9a0b4da6a1a6..b063a53c2c5568f240f77552070103363ea539d2 100644 (file)
@@ -31,6 +31,11 @@ the "cvs add <directory>" call and avoid doing anything in that case?
 >>> should only be built with execv() if the cvs plugin is loaded?
 >>> --[[schmonz]]
 
 >>> should only be built with execv() if the cvs plugin is loaded?
 >>> --[[schmonz]]
 
+>>>> Hadn't considered that. While in wrapper mode the normal getopt is not
+>>>> done, plugin getopt still runs, and so any unsafe options that 
+>>>> other plugins support could be a problem if another user runs
+>>>> the setuid wrapper and passes those options through. --[[Joey]]
+
 > Thing 2 I'm less sure of. (I'd like to see the web UI return
 > immediately on save anyway, to a temporary "rebuilding, please wait
 > if you feel like knowing when it's done" page, but this problem
 > Thing 2 I'm less sure of. (I'd like to see the web UI return
 > immediately on save anyway, to a temporary "rebuilding, please wait
 > if you feel like knowing when it's done" page, but this problem