Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Add TIOCCBRK and TIOCSDTR to the whitelist for pledge ioctl. | 2015-10-16 | 1 | -1/+3 | |
| | | | | | | cu(1) uses these. ok deraadt@ | ||||
* | Pledge support for the parent/resolver in identd(8). | 2015-10-16 | 1 | -1/+14 | |
| | | | | | | | | | | This limits the resolver to just "stdio getpw" or "stdio getpw rpath" depending on whether ~/.noident files are checked. The child/listener cannot use pledge yet because it calls a sysctl that hasn't been whitelisted. "commit" deraadt@ | ||||
* | Pledge for ftp(1) in non-interactive mode. | 2015-10-16 | 1 | -1/+23 | |
| | | | | | | | | We will iterate and remove some of the pledges in the future. This is conservative for now. Tested by sthen@ and myself. ok deraadt@ | ||||
* | Remove -B from EXAMPLES; reminded by jmc@ | 2015-10-16 | 1 | -23/+2 | |
| | |||||
* | Add allocarray(), an overflow-safe allocation function. | 2015-10-16 | 1 | -1/+26 | |
| | | | | | | | | | | We avoided reallocation support because it demands more fancy footwork to deal with the prepended link struct. This has been on my mind for a while, and a 2010 security review of mksh by the Android security team's Chris Palmer suggested it. ok nicm@. Also discussed with millert@ and tedu@. | ||||
* | sync | 2015-10-16 | 2 | -0/+2 | |
| | |||||
* | fine tune the logging some more | 2015-10-16 | 1 | -1/+8 | |
| | |||||
* | simplify logging functions. once a daemon, always a daemon | 2015-10-16 | 1 | -23/+19 | |
| | |||||
* | safety check that we're dealing with the filter we expect | 2015-10-16 | 1 | -2/+5 | |
| | |||||
* | most things should be static | 2015-10-16 | 1 | -15/+15 | |
| | |||||
* | exit(1) is better for the impossible condition | 2015-10-16 | 1 | -2/+3 | |
| | |||||
* | fix some signed/unsigned integer type mismatches in format | 2015-10-15 | 4 | -9/+9 | |
| | | | | strings; reported by Nicholas Lemonias | ||||
* | Do not abuse .Nm for emphasis; | 2015-10-15 | 1 | -12/+5 | |
| | | | | | patch from Michael Reed <m dot reed at mykolab dot com>. Also drop .Tn while here. | ||||
* | Delete two preprocessor constants that are no longer used. | 2015-10-15 | 2 | -7/+2 | |
| | | | | Patch from Michael Reed <m dot reed at mykolab dot com>. | ||||
* | argument to sshkey_from_private() and sshkey_demote() can't be NULL | 2015-10-15 | 1 | -7/+3 | |
| | |||||
* | After spawning, the parent can pledge "stdio rpath wpath cpath" | 2015-10-15 | 1 | -1/+4 | |
| | | | | from rob pierce | ||||
* | Remove three distracting aliases for NULL. | 2015-10-15 | 2 | -15/+11 | |
| | | | | ok nicm@ | ||||
* | Simplify the part of args() that is handling .Bl -column phrases: | 2015-10-15 | 1 | -73/+28 | |
| | | | | | | | | | | Delete manual "Ta" handling because macro handling should not be done in an argument parser but should be left to the macro parsers, which exist anyway and work well. No functional change, minus 40 lines of code. Confusing and redundant code found while investigating an old bug report from tim@. | ||||
* | When blk_full() handles an .It line in .Bl -column and indirectly | 2015-10-15 | 1 | -1/+6 | |
| | | | | | | | | calls phrase_ta() to handle a .Ta child macro, advance the body pointer accordingly, such that a subsequent tab character rewinds the right body block and doesn't fail an assertion. That happened when there was nothing between the .Ta and the tab character. Bug reported by tim@ some time ago. | ||||
* | it is perhaps better style to not call close() on -1, even if harmless | 2015-10-15 | 1 | -6/+12 | |
| | |||||
* | make sure req is zeroed in tcp case | 2015-10-15 | 1 | -2/+2 | |
| | |||||
* | better memory handling of the request/cache chain | 2015-10-15 | 1 | -9/+23 | |
| | |||||
* | do not insert entry into cache until it's fully formed | 2015-10-15 | 1 | -2/+2 | |
| | |||||
* | doh, not all requests are the same size. check len first. | 2015-10-15 | 1 | -2/+3 | |
| | |||||
* | assert is the wrong tool | 2015-10-15 | 1 | -3/+4 | |
| | |||||
* | the inet sockets don't work well with inet6 addrs. pick family from addr. | 2015-10-15 | 1 | -3/+3 | |
| | | | | detected by naddy | ||||
* | everybody can build rebound | 2015-10-15 | 1 | -2/+3 | |
| | |||||
* | no mail for _rebound. deraadt | 2015-10-15 | 1 | -1/+2 | |
| | |||||
* | introduce logerr, since most logging is followed by exit | 2015-10-15 | 1 | -33/+36 | |
| | |||||
* | don't allow NSD to pick up libevent from /usr/local if the libevent2 | 2015-10-15 | 1 | -1/+2 | |
| | | | | package is installed. | ||||
* | now with _rebound user, we can try a little harder at privdrop | 2015-10-15 | 1 | -3/+11 | |
| | |||||
* | _rebound user and group (52) | 2015-10-15 | 2 | -0/+2 | |
| | |||||
* | make the HUP interlocking in the parent work better. | 2015-10-15 | 1 | -10/+24 | |
| | |||||
* | trivial KNF | 2015-10-15 | 1 | -3/+5 | |
| | |||||
* | In syslogd replace the dprintf() macro with a logdebug() function | 2015-10-15 | 3 | -85/+97 | |
| | | | | | as dprintf(3) is in libc now and does something different. OK guenther@ | ||||
* | Pledge login_token with "stdio rpath wpath cpath fattr getpw tty". | 2015-10-15 | 1 | -1/+4 | |
| | | | | OK deraadt@ | ||||
* | just a space in usage, from deraadt | 2015-10-15 | 1 | -2/+2 | |
| | |||||
* | Remove an unused included header (sys/stat.h). | 2015-10-15 | 1 | -2/+1 | |
| | |||||
* | add a hint about the config file, until it changes | 2015-10-15 | 1 | -1/+5 | |
| | |||||
* | child can be pledged down a bit to just sockets and io | 2015-10-15 | 1 | -1/+6 | |
| | |||||
* | When using a pf rule with both nat-to and rdr-to, it could happen | 2015-10-15 | 1 | -3/+6 | |
| | | | | | | | that the nated source port was reused as destination port. Do not initialize nport at the beginning of the function, but where it is needed. OK sashan@ | ||||
* | import rebound, a lightweight dns proxy, for further polishing | 2015-10-15 | 4 | -0/+648 | |
| | |||||
* | Introduce an unsigned char variable for the ctype function calls. | 2015-10-15 | 1 | -5/+7 | |
| | | | | ok millert@ | ||||
* | Don't Xr flock, since that is not the locking method used. | 2015-10-15 | 1 | -3/+2 | |
| | | | | ok millert | ||||
* | Remove disklabel -B (NUMBOOT) support. All the platforms which used to need | 2015-10-15 | 4 | -320/+8 | |
| | | | | | | it are now using MI installboot for that purpose. ok krw@ deraadt@ | ||||
* | Avoid a race between fopen(3) and fchmod(2). Use umask(2) and | 2015-10-15 | 1 | -3/+4 | |
| | | | | | | | | unlink(2) and fopen(3) to prevent an attacker to open an old file with wrong permissions before the secret is written into it. This also guarantees that a new file with correct permissions is created. Without fchmod(2) "fattr" can be removed from pledge. with and OK deraadt@ | ||||
* | No need to create links for xxboot now that MI installboot is the preferred | 2015-10-15 | 1 | -4/+1 | |
| | | | | way to install boot blocks. | ||||
* | Use MI installboot instead of disklabel -B to install boot blocks. | 2015-10-15 | 6 | -36/+19 | |
| | | | | ok krw@ deraadt@ | ||||
* | Add an extra argument to bootstrap() to allow for a limited overlap between an | 2015-10-15 | 6 | -14/+31 | |
| | | | | | | | | | | | | | | | | existing partition and the boot blocks span, and update all callers to require an overlap limit of zero sectors (thus not changing their behaviour). Then, add proper support for vax: copy the 2nd-stage boot block to /boot and install the 1st-stage boot block at the beginning of the disk, retaining the disklabel; allow for an overlap of up to 16 sectors, which is perfectly fine as long as your `a' partition is FFS. Note that regular installs will not even have such an overlap, because the default OpenBSD span on a disk on vax starts at sector 16, but installation media use sperific layout which require this. ok krw@ | ||||
* | add missing comma and missing range restriction, found by smilint | 2015-10-15 | 1 | -4/+7 | |
| |