blob: 2b6604214482c2a5aa051fa43add96220b48f8b8 [file] [log] [blame]
Elly Jonesdd3e8512012-01-23 15:13:38 -05001/*
2 * Copyright (c) 2012 The Chromium OS Authors. All rights reserved.
Elly Jonescd7a9042011-07-22 13:56:51 -04003 * Use of this source code is governed by a BSD-style license that can be
Will Drewry32ac9f52011-08-18 21:36:27 -05004 * found in the LICENSE file.
5 */
Elly Jonescd7a9042011-07-22 13:56:51 -04006
7#define _BSD_SOURCE
8#define _GNU_SOURCE
Jorge Lucangeli Obesc2c9bcc2012-05-01 09:30:24 -07009
Jorge Lucangeli Obes524c0402012-01-17 11:30:23 -080010#include <asm/unistd.h>
Will Drewry32ac9f52011-08-18 21:36:27 -050011#include <ctype.h>
Elly Jonescd7a9042011-07-22 13:56:51 -040012#include <errno.h>
13#include <grp.h>
14#include <inttypes.h>
Will Drewryfe4a3722011-09-16 14:50:50 -050015#include <limits.h>
Elly Jonescd7a9042011-07-22 13:56:51 -040016#include <linux/capability.h>
17#include <linux/securebits.h>
18#include <pwd.h>
19#include <sched.h>
20#include <signal.h>
Will Drewry2f54b6a2011-09-16 13:45:31 -050021#include <stdarg.h>
Jorge Lucangeli Obes524c0402012-01-17 11:30:23 -080022#include <stddef.h>
Elly Jonescd7a9042011-07-22 13:56:51 -040023#include <stdio.h>
24#include <stdlib.h>
25#include <string.h>
26#include <syscall.h>
27#include <sys/capability.h>
28#include <sys/mount.h>
Will Drewryf89aef52011-09-16 16:48:57 -050029#include <sys/param.h>
Elly Jonescd7a9042011-07-22 13:56:51 -040030#include <sys/prctl.h>
Jorge Lucangeli Obes524c0402012-01-17 11:30:23 -080031#include <sys/user.h>
Elly Jonescd7a9042011-07-22 13:56:51 -040032#include <sys/wait.h>
33#include <syslog.h>
34#include <unistd.h>
35
36#include "libminijail.h"
Will Drewry32ac9f52011-08-18 21:36:27 -050037#include "libsyscalls.h"
Elly Jonescd7a9042011-07-22 13:56:51 -040038#include "libminijail-private.h"
39
Jorge Lucangeli Obes524c0402012-01-17 11:30:23 -080040#include "syscall_filter.h"
41
Will Drewry32ac9f52011-08-18 21:36:27 -050042/* Until these are reliably available in linux/prctl.h */
Jorge Lucangeli Obes524c0402012-01-17 11:30:23 -080043#ifndef PR_SET_SECCOMP
Jorge Lucangeli Obesc2c9bcc2012-05-01 09:30:24 -070044# define PR_SET_SECCOMP 22
Jorge Lucangeli Obes524c0402012-01-17 11:30:23 -080045#endif
46
47/* For seccomp_filter using BPF. */
48#ifndef PR_SET_NO_NEW_PRIVS
49# define PR_SET_NO_NEW_PRIVS 38
50#endif
51#ifndef SECCOMP_MODE_FILTER
52# define SECCOMP_MODE_FILTER 2 /* uses user-supplied filter. */
Will Drewry32ac9f52011-08-18 21:36:27 -050053#endif
54
Will Drewry32ac9f52011-08-18 21:36:27 -050055#define die(_msg, ...) do { \
Elly Jonese1749eb2011-10-07 13:54:59 -040056 syslog(LOG_ERR, "libminijail: " _msg, ## __VA_ARGS__); \
57 abort(); \
Will Drewry32ac9f52011-08-18 21:36:27 -050058} while (0)
Elly Jonescd7a9042011-07-22 13:56:51 -040059
Will Drewry32ac9f52011-08-18 21:36:27 -050060#define pdie(_msg, ...) \
Elly Jonese1749eb2011-10-07 13:54:59 -040061 die(_msg ": %s", ## __VA_ARGS__, strerror(errno))
Will Drewry32ac9f52011-08-18 21:36:27 -050062
63#define warn(_msg, ...) \
Elly Jonese1749eb2011-10-07 13:54:59 -040064 syslog(LOG_WARNING, "libminijail: " _msg, ## __VA_ARGS__)
Elly Jonescd7a9042011-07-22 13:56:51 -040065
Elly Jones51a5b6c2011-10-12 19:09:26 -040066struct binding {
67 char *src;
68 char *dest;
69 int writeable;
70 struct binding *next;
71};
72
Will Drewryf89aef52011-09-16 16:48:57 -050073struct minijail {
Elly Jonese1749eb2011-10-07 13:54:59 -040074 struct {
75 int uid:1;
76 int gid:1;
77 int caps:1;
78 int vfs:1;
79 int pids:1;
80 int seccomp:1;
81 int readonly:1;
82 int usergroups:1;
83 int ptrace:1;
Jorge Lucangeli Obesc2c9bcc2012-05-01 09:30:24 -070084 int no_new_privs:1;
Elly Jonese1749eb2011-10-07 13:54:59 -040085 int seccomp_filter:1;
Elly Jones51a5b6c2011-10-12 19:09:26 -040086 int chroot:1;
Elly Jonese1749eb2011-10-07 13:54:59 -040087 } flags;
88 uid_t uid;
89 gid_t gid;
90 gid_t usergid;
91 char *user;
92 uint64_t caps;
93 pid_t initpid;
Jorge Lucangeli Obes524c0402012-01-17 11:30:23 -080094 int filter_len;
Elly Jones51a5b6c2011-10-12 19:09:26 -040095 int binding_count;
96 char *chrootdir;
Jorge Lucangeli Obes524c0402012-01-17 11:30:23 -080097 struct sock_fprog *filter_prog;
Elly Jones51a5b6c2011-10-12 19:09:26 -040098 struct binding *bindings_head;
99 struct binding *bindings_tail;
Will Drewryf89aef52011-09-16 16:48:57 -0500100};
101
Will Drewry6ac91122011-10-21 16:38:58 -0500102struct minijail API *minijail_new(void)
Elly Jonese1749eb2011-10-07 13:54:59 -0400103{
Elly Jones51a5b6c2011-10-12 19:09:26 -0400104 return calloc(1, sizeof(struct minijail));
Elly Jonescd7a9042011-07-22 13:56:51 -0400105}
106
Will Drewry6ac91122011-10-21 16:38:58 -0500107void API minijail_change_uid(struct minijail *j, uid_t uid)
Elly Jonese1749eb2011-10-07 13:54:59 -0400108{
109 if (uid == 0)
110 die("useless change to uid 0");
111 j->uid = uid;
112 j->flags.uid = 1;
Elly Jonescd7a9042011-07-22 13:56:51 -0400113}
114
Will Drewry6ac91122011-10-21 16:38:58 -0500115void API minijail_change_gid(struct minijail *j, gid_t gid)
Elly Jonese1749eb2011-10-07 13:54:59 -0400116{
117 if (gid == 0)
118 die("useless change to gid 0");
119 j->gid = gid;
120 j->flags.gid = 1;
Elly Jonescd7a9042011-07-22 13:56:51 -0400121}
122
Will Drewry6ac91122011-10-21 16:38:58 -0500123int API minijail_change_user(struct minijail *j, const char *user)
Elly Jonese1749eb2011-10-07 13:54:59 -0400124{
125 char *buf = NULL;
126 struct passwd pw;
127 struct passwd *ppw = NULL;
128 ssize_t sz = sysconf(_SC_GETPW_R_SIZE_MAX);
129 if (sz == -1)
130 sz = 65536; /* your guess is as good as mine... */
Elly Joneseb300c52011-09-22 14:35:43 -0400131
Elly Jonesdd3e8512012-01-23 15:13:38 -0500132 /*
133 * sysconf(_SC_GETPW_R_SIZE_MAX), under glibc, is documented to return
Elly Jonese1749eb2011-10-07 13:54:59 -0400134 * the maximum needed size of the buffer, so we don't have to search.
135 */
136 buf = malloc(sz);
137 if (!buf)
138 return -ENOMEM;
139 getpwnam_r(user, &pw, buf, sz, &ppw);
Elly Jonesdd3e8512012-01-23 15:13:38 -0500140 /*
141 * We're safe to free the buffer here. The strings inside pw point
142 * inside buf, but we don't use any of them; this leaves the pointers
143 * dangling but it's safe. ppw points at pw if getpwnam_r succeeded.
144 */
Elly Jonese1749eb2011-10-07 13:54:59 -0400145 free(buf);
146 if (!ppw)
147 return -errno;
148 minijail_change_uid(j, ppw->pw_uid);
149 j->user = strdup(user);
150 if (!j->user)
151 return -ENOMEM;
152 j->usergid = ppw->pw_gid;
153 return 0;
Elly Jonescd7a9042011-07-22 13:56:51 -0400154}
155
Will Drewry6ac91122011-10-21 16:38:58 -0500156int API minijail_change_group(struct minijail *j, const char *group)
Elly Jonese1749eb2011-10-07 13:54:59 -0400157{
158 char *buf = NULL;
159 struct group gr;
160 struct group *pgr = NULL;
161 ssize_t sz = sysconf(_SC_GETGR_R_SIZE_MAX);
162 if (sz == -1)
163 sz = 65536; /* and mine is as good as yours, really */
Elly Joneseb300c52011-09-22 14:35:43 -0400164
Elly Jonesdd3e8512012-01-23 15:13:38 -0500165 /*
166 * sysconf(_SC_GETGR_R_SIZE_MAX), under glibc, is documented to return
Elly Jonese1749eb2011-10-07 13:54:59 -0400167 * the maximum needed size of the buffer, so we don't have to search.
168 */
169 buf = malloc(sz);
170 if (!buf)
171 return -ENOMEM;
172 getgrnam_r(group, &gr, buf, sz, &pgr);
Elly Jonesdd3e8512012-01-23 15:13:38 -0500173 /*
174 * We're safe to free the buffer here. The strings inside gr point
175 * inside buf, but we don't use any of them; this leaves the pointers
176 * dangling but it's safe. pgr points at gr if getgrnam_r succeeded.
177 */
Elly Jonese1749eb2011-10-07 13:54:59 -0400178 free(buf);
179 if (!pgr)
180 return -errno;
181 minijail_change_gid(j, pgr->gr_gid);
182 return 0;
Elly Jonescd7a9042011-07-22 13:56:51 -0400183}
184
Will Drewry6ac91122011-10-21 16:38:58 -0500185void API minijail_use_seccomp(struct minijail *j)
Elly Jonese1749eb2011-10-07 13:54:59 -0400186{
187 j->flags.seccomp = 1;
Elly Jonescd7a9042011-07-22 13:56:51 -0400188}
189
Jorge Lucangeli Obesc2c9bcc2012-05-01 09:30:24 -0700190void API minijail_no_new_privs(struct minijail *j)
191{
192 j->flags.no_new_privs = 1;
193}
194
Will Drewry6ac91122011-10-21 16:38:58 -0500195void API minijail_use_seccomp_filter(struct minijail *j)
Elly Jonese1749eb2011-10-07 13:54:59 -0400196{
Jorge Lucangeli Obes2343d832012-04-25 21:59:48 -0700197 /* TODO(jorgelo): re-enable this when the seccomp BPF merge is done. */
198 j->flags.seccomp_filter = 0;
199}
200
201/* TODO(jorgelo): remove this when the seccomp BPF merge is done. */
202void API minijail_force_seccomp_filter(struct minijail *j)
203{
Elly Jonese1749eb2011-10-07 13:54:59 -0400204 j->flags.seccomp_filter = 1;
Will Drewry32ac9f52011-08-18 21:36:27 -0500205}
206
Will Drewry6ac91122011-10-21 16:38:58 -0500207void API minijail_use_caps(struct minijail *j, uint64_t capmask)
Elly Jonese1749eb2011-10-07 13:54:59 -0400208{
209 j->caps = capmask;
210 j->flags.caps = 1;
Elly Jonescd7a9042011-07-22 13:56:51 -0400211}
212
Will Drewry6ac91122011-10-21 16:38:58 -0500213void API minijail_namespace_vfs(struct minijail *j)
Elly Jonese1749eb2011-10-07 13:54:59 -0400214{
215 j->flags.vfs = 1;
Elly Jonescd7a9042011-07-22 13:56:51 -0400216}
217
Will Drewry6ac91122011-10-21 16:38:58 -0500218void API minijail_namespace_pids(struct minijail *j)
Elly Jonese1749eb2011-10-07 13:54:59 -0400219{
Elly Jonese58176c2012-01-23 11:46:17 -0500220 j->flags.vfs = 1;
221 j->flags.readonly = 1;
Elly Jonese1749eb2011-10-07 13:54:59 -0400222 j->flags.pids = 1;
Elly Jonescd7a9042011-07-22 13:56:51 -0400223}
224
Will Drewry6ac91122011-10-21 16:38:58 -0500225void API minijail_remount_readonly(struct minijail *j)
Elly Jonese1749eb2011-10-07 13:54:59 -0400226{
227 j->flags.vfs = 1;
228 j->flags.readonly = 1;
Elly Jonescd7a9042011-07-22 13:56:51 -0400229}
230
Will Drewry6ac91122011-10-21 16:38:58 -0500231void API minijail_inherit_usergroups(struct minijail *j)
Elly Jonese1749eb2011-10-07 13:54:59 -0400232{
233 j->flags.usergroups = 1;
Elly Jonescd7a9042011-07-22 13:56:51 -0400234}
235
Will Drewry6ac91122011-10-21 16:38:58 -0500236void API minijail_disable_ptrace(struct minijail *j)
Elly Jonese1749eb2011-10-07 13:54:59 -0400237{
238 j->flags.ptrace = 1;
Elly Jonescd7a9042011-07-22 13:56:51 -0400239}
240
Will Drewry6ac91122011-10-21 16:38:58 -0500241int API minijail_enter_chroot(struct minijail *j, const char *dir) {
Elly Jones51a5b6c2011-10-12 19:09:26 -0400242 if (j->chrootdir)
243 return -EINVAL;
244 j->chrootdir = strdup(dir);
245 if (!j->chrootdir)
246 return -ENOMEM;
247 j->flags.chroot = 1;
248 return 0;
249}
250
Will Drewry6ac91122011-10-21 16:38:58 -0500251int API minijail_bind(struct minijail *j, const char *src, const char *dest,
252 int writeable) {
Elly Jones51a5b6c2011-10-12 19:09:26 -0400253 struct binding *b;
254
255 if (*dest != '/')
256 return -EINVAL;
257 b = calloc(1, sizeof(*b));
258 if (!b)
259 return -ENOMEM;
260 b->dest = strdup(dest);
261 if (!b->dest)
262 goto error;
263 b->src = strdup(src);
264 if (!b->src)
265 goto error;
266 b->writeable = writeable;
267
268 syslog(LOG_INFO, "libminijail: bind %s -> %s", src, dest);
269
Elly Jonesdd3e8512012-01-23 15:13:38 -0500270 /*
271 * Force vfs namespacing so the bind mounts don't leak out into the
Elly Jones51a5b6c2011-10-12 19:09:26 -0400272 * containing vfs namespace.
273 */
274 minijail_namespace_vfs(j);
275
276 if (j->bindings_tail)
277 j->bindings_tail->next = b;
278 else
279 j->bindings_head = b;
280 j->bindings_tail = b;
281 j->binding_count++;
282
283 return 0;
284
285error:
286 free(b->src);
287 free(b->dest);
288 free(b);
289 return -ENOMEM;
290}
291
Will Drewry6ac91122011-10-21 16:38:58 -0500292void API minijail_parse_seccomp_filters(struct minijail *j, const char *path)
Elly Jonese1749eb2011-10-07 13:54:59 -0400293{
294 FILE *file = fopen(path, "r");
Jorge Lucangeli Obes524c0402012-01-17 11:30:23 -0800295 if (!file) {
296 pdie("failed to open seccomp filters file '%s'", path);
Elly Jonese1749eb2011-10-07 13:54:59 -0400297 }
Jorge Lucangeli Obes524c0402012-01-17 11:30:23 -0800298
299 struct sock_fprog *fprog = malloc(sizeof(struct sock_fprog));
300 if (compile_filter(file, fprog)) {
301 die("failed to compile seccomp filters BPF program in '%s'", path);
302 }
303
304 j->filter_len = fprog->len;
305 j->filter_prog = fprog;
306
Elly Jonese1749eb2011-10-07 13:54:59 -0400307 fclose(file);
Will Drewry32ac9f52011-08-18 21:36:27 -0500308}
309
Will Drewryf89aef52011-09-16 16:48:57 -0500310struct marshal_state {
Elly Jonese1749eb2011-10-07 13:54:59 -0400311 size_t available;
312 size_t total;
313 char *buf;
Will Drewryf89aef52011-09-16 16:48:57 -0500314};
315
Will Drewry6ac91122011-10-21 16:38:58 -0500316void marshal_state_init(struct marshal_state *state,
317 char *buf, size_t available)
Elly Jonese1749eb2011-10-07 13:54:59 -0400318{
319 state->available = available;
320 state->buf = buf;
321 state->total = 0;
Will Drewryf89aef52011-09-16 16:48:57 -0500322}
323
Will Drewry6ac91122011-10-21 16:38:58 -0500324void marshal_append(struct marshal_state *state,
325 char *src, size_t length)
Elly Jonese1749eb2011-10-07 13:54:59 -0400326{
327 size_t copy_len = MIN(state->available, length);
Will Drewryf89aef52011-09-16 16:48:57 -0500328
Elly Jonese1749eb2011-10-07 13:54:59 -0400329 /* Up to |available| will be written. */
330 if (copy_len) {
331 memcpy(state->buf, src, copy_len);
332 state->buf += copy_len;
333 state->available -= copy_len;
334 }
335 /* |total| will contain the expected length. */
336 state->total += length;
Will Drewryf89aef52011-09-16 16:48:57 -0500337}
338
Will Drewry6ac91122011-10-21 16:38:58 -0500339void minijail_marshal_helper(struct marshal_state *state,
340 const struct minijail *j)
Elly Jonese1749eb2011-10-07 13:54:59 -0400341{
Elly Jones51a5b6c2011-10-12 19:09:26 -0400342 struct binding *b = NULL;
Elly Jonese1749eb2011-10-07 13:54:59 -0400343 marshal_append(state, (char *)j, sizeof(*j));
344 if (j->user)
345 marshal_append(state, j->user, strlen(j->user) + 1);
Elly Jones51a5b6c2011-10-12 19:09:26 -0400346 if (j->chrootdir)
347 marshal_append(state, j->chrootdir, strlen(j->chrootdir) + 1);
Jorge Lucangeli Obes524c0402012-01-17 11:30:23 -0800348 if (j->flags.seccomp_filter && j->filter_prog) {
349 struct sock_fprog *fp = j->filter_prog;
350 marshal_append(state, (char *)fp->filter,
351 fp->len * sizeof(struct sock_filter));
Elly Jonese1749eb2011-10-07 13:54:59 -0400352 }
Elly Jones51a5b6c2011-10-12 19:09:26 -0400353 for (b = j->bindings_head; b; b = b->next) {
354 marshal_append(state, b->src, strlen(b->src) + 1);
355 marshal_append(state, b->dest, strlen(b->dest) + 1);
356 marshal_append(state, (char *)&b->writeable, sizeof(b->writeable));
357 }
Will Drewryf89aef52011-09-16 16:48:57 -0500358}
359
Will Drewry6ac91122011-10-21 16:38:58 -0500360size_t API minijail_size(const struct minijail *j)
Elly Jonese1749eb2011-10-07 13:54:59 -0400361{
362 struct marshal_state state;
363 marshal_state_init(&state, NULL, 0);
364 minijail_marshal_helper(&state, j);
365 return state.total;
Will Drewry2ddaad02011-09-16 11:36:08 -0500366}
367
Elly Jonese1749eb2011-10-07 13:54:59 -0400368int minijail_marshal(const struct minijail *j, char *buf, size_t available)
369{
370 struct marshal_state state;
371 marshal_state_init(&state, buf, available);
372 minijail_marshal_helper(&state, j);
373 return (state.total > available);
Will Drewry2ddaad02011-09-16 11:36:08 -0500374}
375
Elly Jones51a5b6c2011-10-12 19:09:26 -0400376/* consumebytes: consumes @length bytes from a buffer @buf of length @buflength
377 * @length Number of bytes to consume
378 * @buf Buffer to consume from
379 * @buflength Size of @buf
380 *
381 * Returns a pointer to the base of the bytes, or NULL for errors.
382 */
Will Drewry6ac91122011-10-21 16:38:58 -0500383void *consumebytes(size_t length, char **buf, size_t *buflength) {
Elly Jones51a5b6c2011-10-12 19:09:26 -0400384 char *p = *buf;
385 if (length > *buflength)
386 return NULL;
387 *buf += length;
388 *buflength -= length;
389 return p;
390}
391
392/* consumestr: consumes a C string from a buffer @buf of length @length
393 * @buf Buffer to consume
394 * @length Length of buffer
395 *
396 * Returns a pointer to the base of the string, or NULL for errors.
397 */
Will Drewry6ac91122011-10-21 16:38:58 -0500398char *consumestr(char **buf, size_t *buflength) {
Elly Jones51a5b6c2011-10-12 19:09:26 -0400399 size_t len = strnlen(*buf, *buflength);
400 if (len == *buflength)
401 /* There's no null-terminator */
402 return NULL;
403 return consumebytes(len + 1, buf, buflength);
404}
405
Elly Jonese1749eb2011-10-07 13:54:59 -0400406int minijail_unmarshal(struct minijail *j, char *serialized, size_t length)
407{
Elly Jones51a5b6c2011-10-12 19:09:26 -0400408 int i;
409 int count;
Will Drewrybee7ba72011-10-21 20:47:01 -0500410 int ret = -EINVAL;
411
Elly Jonese1749eb2011-10-07 13:54:59 -0400412 if (length < sizeof(*j))
Will Drewrybee7ba72011-10-21 20:47:01 -0500413 goto out;
Elly Jonese1749eb2011-10-07 13:54:59 -0400414 memcpy((void *)j, serialized, sizeof(*j));
415 serialized += sizeof(*j);
416 length -= sizeof(*j);
Will Drewryf89aef52011-09-16 16:48:57 -0500417
Will Drewrybee7ba72011-10-21 20:47:01 -0500418 /* Potentially stale pointers not used as signals. */
419 j->bindings_head = NULL;
420 j->bindings_tail = NULL;
Jorge Lucangeli Obes524c0402012-01-17 11:30:23 -0800421 j->filter_prog = NULL;
Will Drewrybee7ba72011-10-21 20:47:01 -0500422
Elly Jonese1749eb2011-10-07 13:54:59 -0400423 if (j->user) { /* stale pointer */
Elly Jones51a5b6c2011-10-12 19:09:26 -0400424 char *user = consumestr(&serialized, &length);
425 if (!user)
Will Drewrybee7ba72011-10-21 20:47:01 -0500426 goto clear_pointers;
Elly Jones51a5b6c2011-10-12 19:09:26 -0400427 j->user = strdup(user);
Will Drewrybee7ba72011-10-21 20:47:01 -0500428 if (!j->user)
429 goto clear_pointers;
Elly Jonese1749eb2011-10-07 13:54:59 -0400430 }
Will Drewryf89aef52011-09-16 16:48:57 -0500431
Elly Jonesa8d1e1b2011-10-21 15:38:00 -0400432 if (j->chrootdir) { /* stale pointer */
433 char *chrootdir = consumestr(&serialized, &length);
434 if (!chrootdir)
Will Drewrybee7ba72011-10-21 20:47:01 -0500435 goto bad_chrootdir;
Elly Jonesa8d1e1b2011-10-21 15:38:00 -0400436 j->chrootdir = strdup(chrootdir);
Will Drewrybee7ba72011-10-21 20:47:01 -0500437 if (!j->chrootdir)
438 goto bad_chrootdir;
Elly Jonesa8d1e1b2011-10-21 15:38:00 -0400439 }
440
Jorge Lucangeli Obes524c0402012-01-17 11:30:23 -0800441 if (j->flags.seccomp_filter && j->filter_len > 0) {
442 size_t ninstrs = j->filter_len;
443 if (ninstrs > (SIZE_MAX / sizeof(struct sock_filter)) ||
444 ninstrs > USHRT_MAX)
445 goto bad_filters;
446
447 size_t program_len = ninstrs * sizeof(struct sock_filter);
448 void *program = consumebytes(program_len, &serialized, &length);
449 if (!program)
450 goto bad_filters;
451
452 j->filter_prog = malloc(sizeof(struct sock_fprog));
453 j->filter_prog->len = ninstrs;
454 j->filter_prog->filter = malloc(program_len);
455 memcpy(j->filter_prog->filter, program, program_len);
Elly Jonese1749eb2011-10-07 13:54:59 -0400456 }
Elly Jones51a5b6c2011-10-12 19:09:26 -0400457
458 count = j->binding_count;
Elly Jones51a5b6c2011-10-12 19:09:26 -0400459 j->binding_count = 0;
460 for (i = 0; i < count; ++i) {
461 int *writeable;
462 const char *dest;
463 const char *src = consumestr(&serialized, &length);
464 if (!src)
Will Drewrybee7ba72011-10-21 20:47:01 -0500465 goto bad_bindings;
Elly Jones51a5b6c2011-10-12 19:09:26 -0400466 dest = consumestr(&serialized, &length);
467 if (!dest)
Will Drewrybee7ba72011-10-21 20:47:01 -0500468 goto bad_bindings;
Elly Jones51a5b6c2011-10-12 19:09:26 -0400469 writeable = consumebytes(sizeof(*writeable), &serialized, &length);
470 if (!writeable)
Will Drewrybee7ba72011-10-21 20:47:01 -0500471 goto bad_bindings;
Elly Jones51a5b6c2011-10-12 19:09:26 -0400472 if (minijail_bind(j, src, dest, *writeable))
Will Drewrybee7ba72011-10-21 20:47:01 -0500473 goto bad_bindings;
Elly Jones51a5b6c2011-10-12 19:09:26 -0400474 }
475
Elly Jonese1749eb2011-10-07 13:54:59 -0400476 return 0;
Will Drewrybee7ba72011-10-21 20:47:01 -0500477
478bad_bindings:
Jorge Lucangeli Obes524c0402012-01-17 11:30:23 -0800479 if (j->flags.seccomp_filter && j->filter_len > 0) {
480 free(j->filter_prog->filter);
481 free(j->filter_prog);
482 }
Will Drewrybee7ba72011-10-21 20:47:01 -0500483bad_filters:
484 if (j->chrootdir)
485 free(j->chrootdir);
486bad_chrootdir:
487 if (j->user)
488 free(j->user);
489clear_pointers:
490 j->user = NULL;
491 j->chrootdir = NULL;
492out:
493 return ret;
Will Drewry2ddaad02011-09-16 11:36:08 -0500494}
495
Elly Jonese1749eb2011-10-07 13:54:59 -0400496void minijail_preenter(struct minijail *j)
497{
498 /* Strip out options which are minijail_run() only. */
499 j->flags.vfs = 0;
500 j->flags.readonly = 0;
501 j->flags.pids = 0;
Will Drewryfe4a3722011-09-16 14:50:50 -0500502}
503
Elly Jonese1749eb2011-10-07 13:54:59 -0400504void minijail_preexec(struct minijail *j)
505{
506 int vfs = j->flags.vfs;
507 int readonly = j->flags.readonly;
508 if (j->user)
509 free(j->user);
510 j->user = NULL;
511 memset(&j->flags, 0, sizeof(j->flags));
512 /* Now restore anything we meant to keep. */
513 j->flags.vfs = vfs;
514 j->flags.readonly = readonly;
515 /* Note, pidns will already have been used before this call. */
Will Drewry2ddaad02011-09-16 11:36:08 -0500516}
517
Elly Jones51a5b6c2011-10-12 19:09:26 -0400518/* bind_one: Applies bindings from @b for @j, recursing as needed.
519 * @j Minijail these bindings are for
520 * @b Head of list of bindings
521 *
522 * Returns 0 for success.
523 */
Will Drewry6ac91122011-10-21 16:38:58 -0500524int bind_one(const struct minijail *j, struct binding *b) {
Elly Jones51a5b6c2011-10-12 19:09:26 -0400525 int ret = 0;
526 char *dest = NULL;
Elly Jones51a5b6c2011-10-12 19:09:26 -0400527 if (ret)
528 return ret;
529 /* dest has a leading "/" */
530 if (asprintf(&dest, "%s%s", j->chrootdir, b->dest) < 0)
531 return -ENOMEM;
Elly Jonesa1059632011-12-15 15:17:07 -0500532 ret = mount(b->src, dest, NULL, MS_BIND, NULL);
Elly Jones51a5b6c2011-10-12 19:09:26 -0400533 if (ret)
534 pdie("bind: %s -> %s", b->src, dest);
Elly Jonesa1059632011-12-15 15:17:07 -0500535 if (!b->writeable) {
536 ret = mount(b->src, dest, NULL,
537 MS_BIND | MS_REMOUNT | MS_RDONLY, NULL);
538 if (ret)
539 pdie("bind ro: %s -> %s", b->src, dest);
540 }
Elly Jones51a5b6c2011-10-12 19:09:26 -0400541 free(dest);
542 if (b->next)
543 return bind_one(j, b->next);
544 return ret;
545}
546
Will Drewry6ac91122011-10-21 16:38:58 -0500547int enter_chroot(const struct minijail *j) {
Elly Jones51a5b6c2011-10-12 19:09:26 -0400548 int ret;
549 if (j->bindings_head && (ret = bind_one(j, j->bindings_head)))
550 return ret;
551
552 if (chroot(j->chrootdir))
553 return -errno;
554
555 if (chdir("/"))
556 return -errno;
557
558 return 0;
559}
560
Will Drewry6ac91122011-10-21 16:38:58 -0500561int remount_readonly(void)
Elly Jonese1749eb2011-10-07 13:54:59 -0400562{
563 const char *kProcPath = "/proc";
564 const unsigned int kSafeFlags = MS_NODEV | MS_NOEXEC | MS_NOSUID;
Elly Jonesdd3e8512012-01-23 15:13:38 -0500565 /*
566 * Right now, we're holding a reference to our parent's old mount of
Elly Jonese1749eb2011-10-07 13:54:59 -0400567 * /proc in our namespace, which means using MS_REMOUNT here would
568 * mutate our parent's mount as well, even though we're in a VFS
569 * namespace (!). Instead, remove their mount from our namespace
570 * and make our own.
571 */
572 if (umount(kProcPath))
573 return -errno;
574 if (mount("", kProcPath, "proc", kSafeFlags | MS_RDONLY, ""))
575 return -errno;
576 return 0;
Elly Jonescd7a9042011-07-22 13:56:51 -0400577}
578
Will Drewry6ac91122011-10-21 16:38:58 -0500579void drop_caps(const struct minijail *j)
Elly Jonese1749eb2011-10-07 13:54:59 -0400580{
581 cap_t caps = cap_get_proc();
582 cap_value_t raise_flag[1];
583 unsigned int i;
584 if (!caps)
585 die("can't get process caps");
586 if (cap_clear_flag(caps, CAP_INHERITABLE))
587 die("can't clear inheritable caps");
588 if (cap_clear_flag(caps, CAP_EFFECTIVE))
589 die("can't clear effective caps");
590 if (cap_clear_flag(caps, CAP_PERMITTED))
591 die("can't clear permitted caps");
592 for (i = 0; i < sizeof(j->caps) * 8 && cap_valid((int)i); ++i) {
593 if (i != CAP_SETPCAP && !(j->caps & (1 << i)))
594 continue;
595 raise_flag[0] = i;
596 if (cap_set_flag(caps, CAP_EFFECTIVE, 1, raise_flag, CAP_SET))
597 die("can't add effective cap");
598 if (cap_set_flag(caps, CAP_PERMITTED, 1, raise_flag, CAP_SET))
599 die("can't add permitted cap");
600 if (cap_set_flag(caps, CAP_INHERITABLE, 1, raise_flag, CAP_SET))
601 die("can't add inheritable cap");
602 }
603 if (cap_set_proc(caps))
604 die("can't apply cleaned capset");
605 cap_free(caps);
606 for (i = 0; i < sizeof(j->caps) * 8 && cap_valid((int)i); ++i) {
607 if (j->caps & (1 << i))
608 continue;
609 if (prctl(PR_CAPBSET_DROP, i))
610 pdie("prctl(PR_CAPBSET_DROP)");
611 }
Elly Jonescd7a9042011-07-22 13:56:51 -0400612}
613
Will Drewry6ac91122011-10-21 16:38:58 -0500614void API minijail_enter(const struct minijail *j)
Elly Jonese1749eb2011-10-07 13:54:59 -0400615{
616 if (j->flags.pids)
617 die("tried to enter a pid-namespaced jail;"
618 "try minijail_run()?");
Elly Jonescd7a9042011-07-22 13:56:51 -0400619
Elly Jonese1749eb2011-10-07 13:54:59 -0400620 if (j->flags.usergroups && !j->user)
621 die("usergroup inheritance without username");
Elly Jonescd7a9042011-07-22 13:56:51 -0400622
Elly Jonesdd3e8512012-01-23 15:13:38 -0500623 /*
624 * We can't recover from failures if we've dropped privileges partially,
Elly Jonese1749eb2011-10-07 13:54:59 -0400625 * so we don't even try. If any of our operations fail, we abort() the
626 * entire process.
627 */
628 if (j->flags.vfs && unshare(CLONE_NEWNS))
629 pdie("unshare");
Elly Jonescd7a9042011-07-22 13:56:51 -0400630
Elly Jones51a5b6c2011-10-12 19:09:26 -0400631 if (j->flags.chroot && enter_chroot(j))
632 pdie("chroot");
633
Elly Jonese1749eb2011-10-07 13:54:59 -0400634 if (j->flags.readonly && remount_readonly())
635 pdie("remount");
Elly Jonescd7a9042011-07-22 13:56:51 -0400636
Elly Jonese1749eb2011-10-07 13:54:59 -0400637 if (j->flags.caps) {
Elly Jonesdd3e8512012-01-23 15:13:38 -0500638 /*
639 * POSIX capabilities are a bit tricky. If we drop our
Elly Jonese1749eb2011-10-07 13:54:59 -0400640 * capability to change uids, our attempt to use setuid()
641 * below will fail. Hang on to root caps across setuid(), then
642 * lock securebits.
643 */
644 if (prctl(PR_SET_KEEPCAPS, 1))
645 pdie("prctl(PR_SET_KEEPCAPS)");
646 if (prctl
647 (PR_SET_SECUREBITS, SECURE_ALL_BITS | SECURE_ALL_LOCKS))
648 pdie("prctl(PR_SET_SECUREBITS)");
649 }
Elly Jonescd7a9042011-07-22 13:56:51 -0400650
Jorge Lucangeli Obesc2c9bcc2012-05-01 09:30:24 -0700651 /*
652 * Set no_new_privs before installing seccomp filter.
653 * TODO(jorgelo): document call to PR_SET_NO_NEW_PRIVS.
654 */
655 if (j->flags.no_new_privs) {
656 if (prctl(PR_SET_NO_NEW_PRIVS, 1, 0, 0, 0))
657 pdie("prctl(PR_SET_NO_NEW_PRIVS)");
658 }
659
660 /*
661 * Install seccomp filter before dropping root and caps.
662 * WARNING: this means that filter policies *must* allow
663 * setgroups()/setresgid()/setresuid() for dropping root and
664 * capget()/capset()/prctl() for dropping caps.
665 */
666 if (j->flags.seccomp_filter) {
667 if (prctl(PR_SET_SECCOMP, SECCOMP_MODE_FILTER, j->filter_prog))
668 pdie("prctl(PR_SET_SECCOMP, SECCOMP_MODE_FILTER)");
669 }
670
Elly Jonese1749eb2011-10-07 13:54:59 -0400671 if (j->flags.usergroups) {
672 if (initgroups(j->user, j->usergid))
673 pdie("initgroups");
674 } else {
675 /* Only attempt to clear supplemental groups if we are changing
676 * users. */
677 if ((j->uid || j->gid) && setgroups(0, NULL))
678 pdie("setgroups");
679 }
Elly Jonescd7a9042011-07-22 13:56:51 -0400680
Elly Jonese1749eb2011-10-07 13:54:59 -0400681 if (j->flags.gid && setresgid(j->gid, j->gid, j->gid))
682 pdie("setresgid");
Elly Jonescd7a9042011-07-22 13:56:51 -0400683
Elly Jonese1749eb2011-10-07 13:54:59 -0400684 if (j->flags.uid && setresuid(j->uid, j->uid, j->uid))
685 pdie("setresuid");
Elly Jonescd7a9042011-07-22 13:56:51 -0400686
Elly Jonese1749eb2011-10-07 13:54:59 -0400687 if (j->flags.caps)
688 drop_caps(j);
Elly Jonescd7a9042011-07-22 13:56:51 -0400689
Elly Jonesdd3e8512012-01-23 15:13:38 -0500690 /*
691 * seccomp has to come last since it cuts off all the other
Elly Jonese1749eb2011-10-07 13:54:59 -0400692 * privilege-dropping syscalls :)
693 */
Elly Jonese1749eb2011-10-07 13:54:59 -0400694 if (j->flags.seccomp && prctl(PR_SET_SECCOMP, 1))
695 pdie("prctl(PR_SET_SECCOMP)");
Elly Jonescd7a9042011-07-22 13:56:51 -0400696}
697
Will Drewry6ac91122011-10-21 16:38:58 -0500698/* TODO(wad) will visibility affect this variable? */
Elly Jonescd7a9042011-07-22 13:56:51 -0400699static int init_exitstatus = 0;
700
Will Drewry6ac91122011-10-21 16:38:58 -0500701void init_term(int __attribute__ ((unused)) sig)
Elly Jonese1749eb2011-10-07 13:54:59 -0400702{
703 _exit(init_exitstatus);
Elly Jonescd7a9042011-07-22 13:56:51 -0400704}
705
Will Drewry6ac91122011-10-21 16:38:58 -0500706int init(pid_t rootpid)
Elly Jonese1749eb2011-10-07 13:54:59 -0400707{
708 pid_t pid;
709 int status;
710 /* so that we exit with the right status */
711 signal(SIGTERM, init_term);
712 /* TODO(wad) self jail with seccomp_filters here. */
713 while ((pid = wait(&status)) > 0) {
Elly Jonesdd3e8512012-01-23 15:13:38 -0500714 /*
715 * This loop will only end when either there are no processes
Elly Jonese1749eb2011-10-07 13:54:59 -0400716 * left inside our pid namespace or we get a signal.
717 */
718 if (pid == rootpid)
719 init_exitstatus = status;
720 }
721 if (!WIFEXITED(init_exitstatus))
722 _exit(MINIJAIL_ERR_INIT);
723 _exit(WEXITSTATUS(init_exitstatus));
Elly Jonescd7a9042011-07-22 13:56:51 -0400724}
725
Will Drewry6ac91122011-10-21 16:38:58 -0500726int API minijail_from_fd(int fd, struct minijail *j)
Elly Jonese1749eb2011-10-07 13:54:59 -0400727{
728 size_t sz = 0;
729 size_t bytes = read(fd, &sz, sizeof(sz));
730 char *buf;
731 int r;
732 if (sizeof(sz) != bytes)
733 return -EINVAL;
734 if (sz > USHRT_MAX) /* Arbitrary sanity check */
735 return -E2BIG;
736 buf = malloc(sz);
737 if (!buf)
738 return -ENOMEM;
739 bytes = read(fd, buf, sz);
740 if (bytes != sz) {
741 free(buf);
742 return -EINVAL;
743 }
744 r = minijail_unmarshal(j, buf, sz);
745 free(buf);
746 return r;
Will Drewry2f54b6a2011-09-16 13:45:31 -0500747}
748
Will Drewry6ac91122011-10-21 16:38:58 -0500749int API minijail_to_fd(struct minijail *j, int fd)
Elly Jonese1749eb2011-10-07 13:54:59 -0400750{
751 char *buf;
752 size_t sz = minijail_size(j);
753 ssize_t written;
754 int r;
Elly Jonescd7a9042011-07-22 13:56:51 -0400755
Elly Jonese1749eb2011-10-07 13:54:59 -0400756 if (!sz)
757 return -EINVAL;
758 buf = malloc(sz);
759 r = minijail_marshal(j, buf, sz);
760 if (r) {
761 free(buf);
762 return r;
763 }
764 /* Sends [size][minijail]. */
765 written = write(fd, &sz, sizeof(sz));
766 if (written != sizeof(sz)) {
767 free(buf);
768 return -EFAULT;
769 }
770 written = write(fd, buf, sz);
771 if (written < 0 || (size_t) written != sz) {
772 free(buf);
773 return -EFAULT;
774 }
775 free(buf);
776 return 0;
Will Drewry2f54b6a2011-09-16 13:45:31 -0500777}
Elly Jonescd7a9042011-07-22 13:56:51 -0400778
Will Drewry6ac91122011-10-21 16:38:58 -0500779int setup_preload(void)
Elly Jonese1749eb2011-10-07 13:54:59 -0400780{
781 char *oldenv = getenv(kLdPreloadEnvVar) ? : "";
782 char *newenv = malloc(strlen(oldenv) + 2 + strlen(PRELOADPATH));
783 if (!newenv)
784 return -ENOMEM;
Elly Jonescd7a9042011-07-22 13:56:51 -0400785
Elly Jonese1749eb2011-10-07 13:54:59 -0400786 /* Only insert a separating space if we have something to separate... */
787 sprintf(newenv, "%s%s%s", oldenv, strlen(oldenv) ? " " : "",
788 PRELOADPATH);
Elly Jonescd7a9042011-07-22 13:56:51 -0400789
Elly Jonese1749eb2011-10-07 13:54:59 -0400790 /* setenv() makes a copy of the string we give it */
791 setenv(kLdPreloadEnvVar, newenv, 1);
792 free(newenv);
793 return 0;
Elly Jonescd7a9042011-07-22 13:56:51 -0400794}
795
Will Drewry6ac91122011-10-21 16:38:58 -0500796int setup_pipe(int fds[2])
Elly Jonese1749eb2011-10-07 13:54:59 -0400797{
798 int r = pipe(fds);
799 char fd_buf[11];
800 if (r)
801 return r;
802 r = snprintf(fd_buf, sizeof(fd_buf), "%d", fds[0]);
803 if (r <= 0)
804 return -EINVAL;
805 setenv(kFdEnvVar, fd_buf, 1);
806 return 0;
Will Drewryf89aef52011-09-16 16:48:57 -0500807}
808
Will Drewry6ac91122011-10-21 16:38:58 -0500809int API minijail_run(struct minijail *j, const char *filename,
810 char *const argv[])
Elly Jonese1749eb2011-10-07 13:54:59 -0400811{
Jorge Lucangeli Obes9807d032012-04-17 13:36:00 -0700812 return minijail_run_pid(j, filename, argv, NULL);
813}
814
815int API minijail_run_pid(struct minijail *j, const char *filename,
816 char *const argv[], pid_t *pchild_pid)
817{
Elly Jonese1749eb2011-10-07 13:54:59 -0400818 unsigned int pidns = j->flags.pids ? CLONE_NEWPID : 0;
819 char *oldenv, *oldenv_copy = NULL;
820 pid_t child_pid;
821 int pipe_fds[2];
822 int ret;
Ben Chan541c7e52011-08-26 14:55:53 -0700823
Elly Jonese1749eb2011-10-07 13:54:59 -0400824 oldenv = getenv(kLdPreloadEnvVar);
825 if (oldenv) {
826 oldenv_copy = strdup(oldenv);
827 if (!oldenv_copy)
828 return -ENOMEM;
829 }
Will Drewryf89aef52011-09-16 16:48:57 -0500830
Elly Jonese1749eb2011-10-07 13:54:59 -0400831 if (setup_preload())
832 return -EFAULT;
Will Drewry2f54b6a2011-09-16 13:45:31 -0500833
Elly Jonesdd3e8512012-01-23 15:13:38 -0500834 /*
835 * Before we fork(2) and execve(2) the child process, we need to open
Elly Jonese1749eb2011-10-07 13:54:59 -0400836 * a pipe(2) to send the minijail configuration over.
837 */
838 if (setup_pipe(pipe_fds))
839 return -EFAULT;
Elly Jonescd7a9042011-07-22 13:56:51 -0400840
Elly Jonese1749eb2011-10-07 13:54:59 -0400841 child_pid = syscall(SYS_clone, pidns | SIGCHLD, NULL);
842 if (child_pid < 0) {
843 free(oldenv_copy);
844 return child_pid;
845 }
Will Drewryf89aef52011-09-16 16:48:57 -0500846
Elly Jonese1749eb2011-10-07 13:54:59 -0400847 if (child_pid) {
848 /* Restore parent's LD_PRELOAD. */
849 if (oldenv_copy) {
850 setenv(kLdPreloadEnvVar, oldenv_copy, 1);
851 free(oldenv_copy);
852 } else {
853 unsetenv(kLdPreloadEnvVar);
854 }
855 unsetenv(kFdEnvVar);
856 j->initpid = child_pid;
857 close(pipe_fds[0]); /* read endpoint */
858 ret = minijail_to_fd(j, pipe_fds[1]);
859 close(pipe_fds[1]); /* write endpoint */
860 if (ret) {
861 kill(j->initpid, SIGKILL);
862 die("failed to send marshalled minijail");
863 }
Jorge Lucangeli Obes9807d032012-04-17 13:36:00 -0700864 if (pchild_pid)
865 *pchild_pid = child_pid;
Elly Jonese1749eb2011-10-07 13:54:59 -0400866 return 0;
867 }
868 free(oldenv_copy);
Ben Chan541c7e52011-08-26 14:55:53 -0700869
Elly Jonese1749eb2011-10-07 13:54:59 -0400870 /* Drop everything that cannot be inherited across execve. */
871 minijail_preexec(j);
872 /* Jail this process and its descendants... */
873 minijail_enter(j);
Elly Jonescd7a9042011-07-22 13:56:51 -0400874
Elly Jonese1749eb2011-10-07 13:54:59 -0400875 if (pidns) {
Elly Jonesdd3e8512012-01-23 15:13:38 -0500876 /*
877 * pid namespace: this process will become init inside the new
Elly Jonese1749eb2011-10-07 13:54:59 -0400878 * namespace, so fork off a child to actually run the program
879 * (we don't want all programs we might exec to have to know
880 * how to be init).
881 */
882 child_pid = fork();
883 if (child_pid < 0)
884 _exit(child_pid);
885 else if (child_pid > 0)
886 init(child_pid); /* never returns */
887 }
Elly Jonescd7a9042011-07-22 13:56:51 -0400888
Elly Jonesdd3e8512012-01-23 15:13:38 -0500889 /*
890 * If we aren't pid-namespaced:
Elly Jonese1749eb2011-10-07 13:54:59 -0400891 * calling process
892 * -> execve()-ing process
893 * If we are:
894 * calling process
895 * -> init()-ing process
896 * -> execve()-ing process
897 */
898 _exit(execve(filename, argv, environ));
Elly Jonescd7a9042011-07-22 13:56:51 -0400899}
900
Will Drewry6ac91122011-10-21 16:38:58 -0500901int API minijail_kill(struct minijail *j)
Elly Jonese1749eb2011-10-07 13:54:59 -0400902{
903 int st;
904 if (kill(j->initpid, SIGTERM))
905 return -errno;
906 if (waitpid(j->initpid, &st, 0) < 0)
907 return -errno;
908 return st;
Elly Jonescd7a9042011-07-22 13:56:51 -0400909}
910
Will Drewry6ac91122011-10-21 16:38:58 -0500911int API minijail_wait(struct minijail *j)
Elly Jonese1749eb2011-10-07 13:54:59 -0400912{
913 int st;
914 if (waitpid(j->initpid, &st, 0) < 0)
915 return -errno;
Jorge Lucangeli Obesc2c9bcc2012-05-01 09:30:24 -0700916 if (!WIFEXITED(st)) {
917 if (WIFSIGNALED(st))
918 warn("child process received signal %d", WTERMSIG(st));
Elly Jonese1749eb2011-10-07 13:54:59 -0400919 return MINIJAIL_ERR_JAIL;
Jorge Lucangeli Obesc2c9bcc2012-05-01 09:30:24 -0700920 }
Elly Jonese1749eb2011-10-07 13:54:59 -0400921 return WEXITSTATUS(st);
Elly Jonescd7a9042011-07-22 13:56:51 -0400922}
923
Will Drewry6ac91122011-10-21 16:38:58 -0500924void API minijail_destroy(struct minijail *j)
Elly Jonese1749eb2011-10-07 13:54:59 -0400925{
Jorge Lucangeli Obes524c0402012-01-17 11:30:23 -0800926 if (j->flags.seccomp_filter && j->filter_prog) {
927 free(j->filter_prog->filter);
928 free(j->filter_prog);
Elly Jonese1749eb2011-10-07 13:54:59 -0400929 }
Elly Jones51a5b6c2011-10-12 19:09:26 -0400930 while (j->bindings_head) {
931 struct binding *b = j->bindings_head;
932 j->bindings_head = j->bindings_head->next;
933 free(b->dest);
934 free(b->src);
935 free(b);
936 }
937 j->bindings_tail = NULL;
Elly Jonese1749eb2011-10-07 13:54:59 -0400938 if (j->user)
939 free(j->user);
Will Drewrybee7ba72011-10-21 20:47:01 -0500940 if (j->chrootdir)
941 free(j->chrootdir);
Elly Jonese1749eb2011-10-07 13:54:59 -0400942 free(j);
Elly Jonescd7a9042011-07-22 13:56:51 -0400943}