CVS: cvs.openbsd.org: src

2019-08-27 Thread Adam Wolk
CVSROOT:/cvs
Module name:src
Changes by: aw...@cvs.openbsd.org   2019/08/27 14:05:02

Modified files:
share/man/man8 : rc.d.8 

Log message:
Sync example with current databases/postgresql flags

OK sthen@, pea@



CVS: cvs.openbsd.org: www

2019-08-27 Thread Adam Wolk
CVSROOT:/cvs
Module name:www
Changes by: aw...@cvs.openbsd.org   2019/08/27 13:57:57

Modified files:
faq: current.html 

Log message:
2019/08/27 - [ports] Datadir added to default postgres flags



CVS: cvs.openbsd.org: src

2017-07-15 Thread Adam Wolk
CVSROOT:/cvs
Module name:src
Changes by: aw...@cvs.openbsd.org   2017/07/15 10:01:14

Modified files:
sbin/pfctl : pfctl.h pfctl_table.c 

Log message:
sbin/pfctl: void functions and exit(3) on error

Changes:
voided:
- pfctl_clear_tables
- pfctl_show_tables
- pfctl_show_ifaces

Those functions now exit(3) in case of error instead
of passing it up to the callers (where it was ignored).

OK mikeb@, sashan@



CVS: cvs.openbsd.org: src

2017-07-15 Thread Adam Wolk
CVSROOT:/cvs
Module name:src
Changes by: aw...@cvs.openbsd.org   2017/07/15 09:59:52

Modified files:
regress/sbin/pfctl: pfcmd1.opts 

Log message:
regress/sbin/pfctl - use -n for pfcmd group

Passing in the -n which just results in the rules being parsed.

The definition for the pfctlcmd testing group in the Makefile is:
# pfcmd: test pfctl command line parsing

This test passeed only because pfctl didn't exit with an error
when it was asked to clear tables from a non existing anchor.

with the -n flag we still test if the anchor command can be combined
with -Fa but don't actually attempt to run the clearing code.

OK mikeb@, sashan@



CVS: cvs.openbsd.org: src

2017-06-16 Thread Adam Wolk
CVSROOT:/cvs
Module name:src
Changes by: aw...@cvs.openbsd.org   2017/06/16 13:59:13

Modified files:
sbin/pfctl : pfctl.c 

Log message:
pfctl: merge identifical if conditions and void functions discarding ret values

Transform the following functions (which never return anything other than 0, and
whose return value is never used) to void:

* pfctl_clear_stats, pfctl_clear_interface_flags, pfctl_clear_rules,
pfctl_clear_src_nodes, pfctl_clear_states
* pfctl_kill_src_nodes, pfctl_net_kill_states, pfctl_label_kill_states,
pfctl_id_kill_states, pfctl_key_kill_states

inside main: merge two identical if conditions next to each other into one.

Submitted by rain1 (thanks!) with additional credit to:
- mikeb@ for pointing out we can void all _clear_ functions
- ghostyy for pointing out all _kill_ functions can be voided

OK tb@, mikeb@



CVS: cvs.openbsd.org: src

2017-06-11 Thread Adam Wolk
CVSROOT:/cvs
Module name:src
Changes by: aw...@cvs.openbsd.org   2017/06/11 11:32:19

Modified files:
usr.bin/ktrace : ktrace.c 

Log message:
usr/bin/ktrace: replace snprintf(3)/write(2) with dprintf(3)

Brought to attention by BlackFrog on #openbsd-daily

OK deraadt@



CVS: cvs.openbsd.org: src

2017-06-07 Thread Adam Wolk
CVSROOT:/cvs
Module name:src
Changes by: aw...@cvs.openbsd.org   2017/06/07 03:11:52

Modified files:
usr.bin/htpasswd: htpasswd.c 

Log message:
htpasswd: use crypt_newhash instead of the bcrypt API

man bcrypt states:
These functions are deprecated in favor of crypt_checkpass(3) and
crypt_newhash(3).

hence with this change we move htpasswd to the new API, while here
also change the rounds from a hardcoded 8 to automatic selection based
on system performance.

OK florian@



CVS: cvs.openbsd.org: src

2017-05-28 Thread Adam Wolk
CVSROOT:/cvs
Module name:src
Changes by: aw...@cvs.openbsd.org   2017/05/28 02:07:33

Modified files:
etc: netstart 

Log message:
etc/netstart: use colon separator instead of dot with chown

OK jung@, deraadt@, jmc@



CVS: cvs.openbsd.org: src

2017-05-28 Thread Adam Wolk
CVSROOT:/cvs
Module name:src
Changes by: aw...@cvs.openbsd.org   2017/05/28 02:03:36

Modified files:
bin/chmod  : Makefile chmod.c chown.8 

Log message:
chown: Remove SUPPORT_DOT ifdef - it's on by default for 22 years

The old syntax was deprecated 25 years ago when the utility was
first standardised in IEEE Std 1003.2-1992 ("POSIX.2"). There was
no POSIX version of chown with the dot separator.

Let's stop pretending that it will ever go away.

OK jung@, deraadt@, jmc@



CVS: cvs.openbsd.org: www

2016-12-06 Thread Adam Wolk
CVSROOT:/cvs
Module name:www
Changes by: aw...@cvs.openbsd.org   2016/12/06 13:41:33

Modified files:
faq: faq15.html 

Log message:
Point pkg_sign(1) man link at 6.0 release

pkg_sign -s syntax changed between 6.0 and -current
point the man page at the version in sync with the FAQ
content and mark it with XXXrelease to be reverted
for the 6.1 FAQ post release.

Pointed out by: _gypcio on IRC Freenode

OK tj@



CVS: cvs.openbsd.org: src

2016-09-15 Thread Adam Wolk
CVSROOT:/cvs
Module name:src
Changes by: aw...@cvs.openbsd.org   2016/09/15 11:53:38

Modified files:
share/man/man9 : Makefile 
Added files:
share/man/man9 : usb_add_task.9 

Log message:
Document the USB task API

OK mpi@

change post OK:
- long lines in the source file wrapped
- added note about USB_TASK_TYPE_ABORT from mpi@



CVS: cvs.openbsd.org: www

2016-06-06 Thread Adam Wolk
CVSROOT:/cvs
Module name:www
Changes by: aw...@cvs.openbsd.org   2016/06/06 06:35:08

Modified files:
.  : errata58.html errata59.html 

Log message:
libcrypto errata
- 5.8#17
- 5.9#11

OK tj@



CVS: cvs.openbsd.org: src

2016-05-02 Thread Adam Wolk
CVSROOT:/cvs
Module name:src
Changes by: aw...@cvs.openbsd.org   2016/05/02 20:06:54

Modified files:
usr.bin/telnet : commands.c 

Log message:
telnet should not verify if hostname is a fully qualified domain

telnet would perform a gethostbyname on close/quit commands
when the machine hostname is not a fully qualified domain.
This behavior added on 12-March-98 breaks the current pledge
that "dns" operations are not needed after establishing
a connection leading to a coredump from telnet when
quiting or closing an existing connection.

OK chris@