diff options
| author |   mestre <mestre@openbsd.org> | 2018-09-13 07:49:33 +0000 |
|---|---|---|
| committer | mestre <mestre@openbsd.org> | 2018-09-13 07:49:33 +0000 |
| commit | a2104a60048b84a317d027fc90a63c72227ffced (patch) | |
| tree | 06abf8aa3a0117470f4edfc1f6d3fab6b788a69f | |
| parent | Avoid unneeded variable in gen_dynnode() (diff) | |
| download | wireguard-openbsd-a2104a60048b84a317d027fc90a63c72227ffced.tar.xz wireguard-openbsd-a2104a60048b84a317d027fc90a63c72227ffced.zip | |
When unveil(2) was introduced one break from SYS_access case was removed
here, this adds it back. Noticed by Coverity 1471854.
feedback from semarie@ OK deraadt@
| -rw-r--r-- | sys/kern/kern_pledge.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/sys/kern/kern_pledge.c b/sys/kern/kern_pledge.c index be0e31339f4..00ed0b3be67 100644 --- a/sys/kern/kern_pledge.c +++ b/sys/kern/kern_pledge.c @@ -1,4 +1,4 @@ -/* $OpenBSD: kern_pledge.c,v 1.242 2018/08/20 10:00:04 kettenis Exp $ */ +/* $OpenBSD: kern_pledge.c,v 1.243 2018/09/13 07:49:33 mestre Exp $ */ /* * Copyright (c) 2015 Nicholas Marriott <nicm@openbsd.org> @@ -623,6 +623,7 @@ pledge_namei(struct proc *p, struct nameidata *ni, char *origpath) } else return (pledge_fail(p, error, PLEDGE_GETPW)); } + break; case SYS_open: /* daemon(3) or other such functions */ if ((ni->ni_pledge & ~(PLEDGE_RPATH | PLEDGE_WPATH)) == 0 && |