diff options
| author | Ingo Schwarze <schwarze@openbsd.org> | 2015-12-15 17:38:45 +0000 |
|---|---|---|
| committer | Ingo Schwarze <schwarze@openbsd.org> | 2015-12-15 17:38:45 +0000 |
| commit | f0d907b5ddcf8c6632e2f8c3301b9ef19abc0a02 (patch) | |
| tree | ff9a80d2c3722497da3f61fc232083562c4f7945 /main.c | |
| parent | cd024781e6da76dda541a980fc161f031b33e300 (diff) | |
| download | mandoc-f0d907b5ddcf8c6632e2f8c3301b9ef19abc0a02.tar.gz | |
pledge(2) style:
Make sure to always use the idiom 'if (pledge("'
such that it can easily be searched for.
No functional change.
Requested by deraadt@ some time ago.
Diffstat (limited to 'main.c')
| -rw-r--r-- | main.c | 15 |
1 files changed, 10 insertions, 5 deletions
@@ -299,8 +299,9 @@ main(int argc, char *argv[]) use_pager = 0; #if HAVE_PLEDGE - if (!use_pager && pledge("stdio rpath flock", NULL) == -1) - err((int)MANDOCLEVEL_SYSERR, "pledge"); + if (!use_pager) + if (pledge("stdio rpath flock", NULL) == -1) + err((int)MANDOCLEVEL_SYSERR, "pledge"); #endif /* Parse arguments. */ @@ -430,9 +431,13 @@ main(int argc, char *argv[]) /* mandoc(1) */ #if HAVE_PLEDGE - if (pledge(use_pager ? "stdio rpath tmppath tty proc exec" : - "stdio rpath", NULL) == -1) - err((int)MANDOCLEVEL_SYSERR, "pledge"); + if (use_pager) { + if (pledge("stdio rpath tmppath tty proc exec", NULL) == -1) + err((int)MANDOCLEVEL_SYSERR, "pledge"); + } else { + if (pledge("stdio rpath", NULL) == -1) + err((int)MANDOCLEVEL_SYSERR, "pledge"); + } #endif if (search.argmode == ARG_FILE && ! moptions(&options, auxpaths)) |