source: server/common/patches/httpd-suexec-scripts.patch @ 298

Last change on this file since 298 was 298, checked in by andersk, 17 years ago
* Call static-cat directly from suexec on static extensions, instead of going through binfmt_misc. This will soon be replaced with a more permanent solution. * Fix a one-character buffer overflow in the suexec patch.
File size: 5.9 KB
RevLine 
[1]1# scripts.mit.edu httpd suexec patch
[103]2# Copyright (C) 2006  Jeff Arnold <jbarnold@mit.edu>, Joe Presbrey <presbrey@mit.edu>
[1]3#
4# This program is free software; you can redistribute it and/or
5# modify it under the terms of the GNU General Public License
6# as published by the Free Software Foundation; either version 2
7# of the License, or (at your option) any later version.
8#
9# This program is distributed in the hope that it will be useful,
10# but WITHOUT ANY WARRANTY; without even the implied warranty of
11# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
12# GNU General Public License for more details.
13#
14# You should have received a copy of the GNU General Public License
15# along with this program; if not, write to the Free Software
16# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, USA
17#
18# See /COPYRIGHT in this repository for more information.
19#
[103]20--- httpd-2.2.2/support/Makefile.in.old 2005-07-06 19:15:34.000000000 -0400
21+++ httpd-2.2.2/support/Makefile.in     2007-01-20 17:12:51.000000000 -0500
22@@ -60,7 +60,7 @@
23
24 suexec_OBJECTS = suexec.lo
25 suexec: $(suexec_OBJECTS)
26-       $(LINK) $(suexec_OBJECTS)
27+       $(LINK) -lselinux $(suexec_OBJECTS)
28
29 htcacheclean_OBJECTS = htcacheclean.lo
30 htcacheclean: $(htcacheclean_OBJECTS)
[1]31--- httpd-2.2.2/support/suexec.c.old    2006-04-21 21:53:06.000000000 -0400
[298]32+++ httpd-2.2.2/support/suexec.c        2007-04-23 03:04:56.000000000 -0400
33@@ -30,6 +30,8 @@
34  *
35  */
36 
37+#define STATIC_CAT_PATH "/usr/local/bin/static-cat"
38+
39 #include "apr.h"
40 #include "ap_config.h"
41 #include "suexec.h"
42@@ -46,6 +48,7 @@
[103]43 #include <stdio.h>
44 #include <stdarg.h>
45 #include <stdlib.h>
46+#include <selinux/selinux.h>
47 
48 #ifdef HAVE_PWD_H
49 #include <pwd.h>
[298]50@@ -95,6 +98,7 @@
[1]51 {
52     /* variable name starts with */
53     "HTTP_",
54+    "HTTPS_",
55     "SSL_",
56 
57     /* variable name is */
[298]58@@ -140,6 +144,7 @@
[1]59     "UNIQUE_ID=",
60     "USER_NAME=",
61     "TZ=",
62+    "PHPRC=",
63     NULL
64 };
65 
[298]66@@ -245,6 +250,52 @@
67     environ = cleanenv;
68 }
69 
70+static const char *static_extensions[] = {
71+    "html",
72+    "css",
73+    "gif",
74+    "jpg",
75+    "png",
76+    "htm",
77+    "jpeg",
78+    "js",
79+    "ico",
80+    "xml",
81+    "xsl",
82+    "tiff",
83+    "tif",
84+    "tgz",
85+    "tar",
86+    "jar",
87+    "zip",
88+    "pdf",
89+    "ps",
90+    "doc",
91+    "xls",
92+    "ppt",
93+    "swf",
94+    "mp3",
95+    "mov",
96+    "wmv",
97+    "mpg",
98+    "mpeg",
99+    "avi",
100+    "il",
101+    "JPG",
102+    NULL
103+};
104+
105+static int is_static_extension(const char *file)
106+{
107+    const char *extension = strrchr(file, '.');
108+    const char **p;
109+    if (extension == NULL) return 0;
110+    for (p = static_extensions; *p; ++p) {
111+        if (strcmp(extension + 1, *p) == 0) return 1;
112+    }
113+    return 0;
114+}
115+
116 int main(int argc, char *argv[])
117 {
118     int userdir = 0;        /* ~userdir flag             */
119@@ -450,7 +501,7 @@
[103]120      * Error out if attempt is made to execute as root or as
121      * a UID less than AP_UID_MIN.  Tsk tsk.
122      */
123-    if ((uid == 0) || (uid < AP_UID_MIN)) {
124+    if ((uid == 0) || (uid < AP_UID_MIN && uid != 102)) {
125         log_err("cannot run as forbidden uid (%d/%s)\n", uid, cmd);
126         exit(107);
127     }
[298]128@@ -482,6 +533,17 @@
[103]129         log_err("failed to setuid (%ld: %s)\n", uid, cmd);
130         exit(110);
131     }
132+    if (uid == 102) {
[113]133+        if (setexeccon("system_u:system_r:signup_t:s0") == -1) {
[204]134+            log_err("failed to setexeccon (%ld: %s) to signup_t\n", uid, cmd);
[103]135+            exit(201);
136+        }
137+    } else {
138+        if (setexeccon("user_u:user_r:user_t:s0") == -1) {
[204]139+            log_err("failed to setexeccon (%ld: %s) to user_t\n", uid, cmd);
[103]140+            exit(202);
141+        }
142+    }
143 
144     /*
145      * Get the current working directory, as well as the proper
[298]146@@ -513,6 +575,13 @@
[1]147             exit(113);
148         }
149     }
[298]150+    size_t expected_len = strlen(target_homedir)+1+strlen(AP_USERDIR_SUFFIX)+1;
151+    char *expected = malloc(expected_len);
152+    snprintf(expected, expected_len, "%s/%s", target_homedir, AP_USERDIR_SUFFIX);
153+    if (strncmp(cwd, expected, expected_len) != 0) {
[1]154+        log_err("error: file's directory not a subdirectory of user's home directory (%s, %s)\n", cwd, expected);
155+        exit(114);
156+    }
157 
158     if ((strncmp(cwd, dwd, strlen(dwd))) != 0) {
159         log_err("command not in docroot (%s/%s)\n", cwd, cmd);
[298]160@@ -530,15 +598,17 @@
[1]161     /*
162      * Error out if cwd is writable by others.
163      */
164+#if 0
165     if ((dir_info.st_mode & S_IWOTH) || (dir_info.st_mode & S_IWGRP)) {
166         log_err("directory is writable by others: (%s)\n", cwd);
167         exit(116);
168     }
169+#endif
170 
171     /*
172      * Error out if we cannot stat the program.
173      */
174-    if (((lstat(cmd, &prg_info)) != 0) || (S_ISLNK(prg_info.st_mode))) {
175+    if (((lstat(cmd, &prg_info)) != 0) /*|| (S_ISLNK(prg_info.st_mode))*/) {
176         log_err("cannot stat program: (%s)\n", cmd);
177         exit(117);
178     }
[298]179@@ -546,10 +616,12 @@
[1]180     /*
181      * Error out if the program is writable by others.
182      */
183+#if 0
184     if ((prg_info.st_mode & S_IWOTH) || (prg_info.st_mode & S_IWGRP)) {
185         log_err("file is writable by others: (%s/%s)\n", cwd, cmd);
186         exit(118);
187     }
188+#endif
189 
190     /*
191      * Error out if the file is setuid or setgid.
[298]192@@ -563,6 +635,7 @@
[1]193      * Error out if the target name/group is different from
194      * the name/group of the cwd or the program.
195      */
196+#if 0
197     if ((uid != dir_info.st_uid) ||
198         (gid != dir_info.st_gid) ||
199         (uid != prg_info.st_uid) ||
[298]200@@ -574,6 +647,7 @@
[1]201                 prg_info.st_uid, prg_info.st_gid);
202         exit(120);
203     }
204+#endif
205     /*
206      * Error out if the program is not executable for the user.
207      * Otherwise, she won't find any error in the logs except for
[298]208@@ -609,6 +683,13 @@
209         log = NULL;
210     }
211 
212+    if (is_static_extension(cmd)) {
213+        argv[2] = STATIC_CAT_PATH;
214+        execv(STATIC_CAT_PATH, &argv[2]);
215+       log_err("(%d)%s: static_cat exec failed (%s)\n", errno, strerror(errno), argv[2]);
216+       exit(255);
217+    }
218+
219     /*
220      * Execute the command, replacing our image with its own.
221      */
Note: See TracBrowser for help on using the repository browser.