CVS: cvs.openbsd.org: src

2015-02-15 Thread Doug Hogan
CVSROOT:/cvs
Module name:src
Changes by: d...@cvs.openbsd.org2015/02/15 23:48:17

Modified files:
regress/lib/libssl/bytestring: bytestringtest.c 

Log message:
Add more error checking and free resources in bytestringtest.



CVS: cvs.openbsd.org: src

2015-02-15 Thread Alexandre Ratchov
CVSROOT:/cvs
Module name:src
Changes by: ratc...@cvs.openbsd.org 2015/02/15 23:35:17

Modified files:
usr.bin/sndiod : sock.c 

Log message:
Check if slot is NULL, and if so, drop the connection. For now slot
is never NULL, but this is an accident.



CVS: cvs.openbsd.org: src

2015-02-15 Thread Alexandre Ratchov
CVSROOT:/cvs
Module name:src
Changes by: ratc...@cvs.openbsd.org 2015/02/15 23:28:05

Modified files:
usr.bin/sndiod : dev.h siofile.c 

Log message:
add missing protos, fix spacing



CVS: cvs.openbsd.org: src

2015-02-15 Thread Alexandre Ratchov
CVSROOT:/cvs
Module name:src
Changes by: ratc...@cvs.openbsd.org 2015/02/15 23:26:24

Modified files:
usr.bin/sndiod : file.c 

Log message:
Allow files to be skipped in the poll(2) event loop, in case
there are events already available (for immediate processing)



CVS: cvs.openbsd.org: src

2015-02-15 Thread Alexandre Ratchov
CVSROOT:/cvs
Module name:src
Changes by: ratc...@cvs.openbsd.org 2015/02/15 23:11:33

Modified files:
usr.bin/sndiod : abuf.c 

Log message:
Use signed integers as offset in the buffer (as aucat does) to
avoid other signed integer being accidently promoted to unsigned
in future code.



CVS: cvs.openbsd.org: src

2015-02-15 Thread Alexandre Ratchov
CVSROOT:/cvs
Module name:src
Changes by: ratc...@cvs.openbsd.org 2015/02/15 23:07:56

Modified files:
lib/libsndio   : mio_rmidi.c 

Log message:
Drop yet another hook for ':' char in midi device names.



CVS: cvs.openbsd.org: src

2015-02-15 Thread Alexandre Ratchov
CVSROOT:/cvs
Module name:src
Changes by: ratc...@cvs.openbsd.org 2015/02/15 23:04:06

Modified files:
lib/libsndio   : sio_sun.c 

Log message:
Remove backward compat bits to support for ':' character in sndio
device names. Remove wrong comments.



CVS: cvs.openbsd.org: src

2015-02-15 Thread Miod Vallat
CVSROOT:/cvs
Module name:src
Changes by: m...@cvs.openbsd.org2015/02/15 22:23:43

Modified files:
sys/kern   : exec_subr.c 

Log message:
In exec_setup_stack(), move the stack in the right direction if
MACHINE_STACK_GROWS_UP.



CVS: cvs.openbsd.org: src

2015-02-15 Thread Ted Unangst
CVSROOT:/cvs
Module name:src
Changes by: t...@cvs.openbsd.org2015/02/15 18:44:41

Modified files:
bin/ksh: emacs.c 

Log message:
bcmp to memcmp



CVS: cvs.openbsd.org: src

2015-02-15 Thread Doug Hogan
CVSROOT:/cvs
Module name:src
Changes by: d...@cvs.openbsd.org2015/02/15 15:29:02

Modified files:
lib/libssl/src/crypto/dsa: dsa_gen.c 

Log message:
Avoid calling BN_CTX_end() on a context that wasn't started.

In dsa_builtin_paramgen(), if BN_MONT_CTX_new() fails, the BN_CTX_new()
call above it will have allocated a ctx without calling BN_CTX_start() on
it.  The error handling calls BN_CTX_end() when ctx is allocated.

Move the BN_MONT_CTX_new() call up so it will fail first without splitting
up the BN_CTX_new() and BN_CTX_start().

tweak + ok miod@, ok bcook@



CVS: cvs.openbsd.org: src

2015-02-15 Thread Anthony J. Bentley
CVSROOT:/cvs
Module name:src
Changes by: bent...@cvs.openbsd.org 2015/02/15 15:27:51

Modified files:
libexec/spamd  : spamd.8 

Log message:
Don't use Aq macros when <> is intended; they are not the same thing.

ok schwarze@



CVS: cvs.openbsd.org: src

2015-02-15 Thread Anthony J. Bentley
CVSROOT:/cvs
Module name:src
Changes by: bent...@cvs.openbsd.org 2015/02/15 15:26:45

Modified files:
lib/libarch/i386: i386_get_ldt.2 
lib/libz   : compress.3 
sbin/pflogd: pflogd.8 
share/man/man4 : speaker.4 usb.4 wsdisplay.4 
share/man/man4/man4.luna88k: lcd.4 
share/man/man4/man4.sparc: fga.4 
share/man/man4/man4.sparc64: uperf.4 
share/man/man4/man4.vax: mtc.4 ra.4 uda.4 
share/man/man5 : fs.5 
share/man/man9 : ktrace.9 pci_conf_read.9 
usr.bin/file   : file.1 
usr.bin/lex: flex.1 
usr.bin/openssl: openssl.1 

Log message:
Use "In" to mark up include files, instead of wrongly wrapping with Aq.

Aq is not the same as <> in non-ASCII situations, so this caused
incorrect output in some places. And it provided no semantics besides.

ok schwarze@



CVS: cvs.openbsd.org: src

2015-02-15 Thread Todd C. Miller
CVSROOT:/cvs
Module name:src
Changes by: mill...@cvs.openbsd.org 2015/02/15 15:18:29

Modified files:
bin/pax: extern.h file_subs.c tables.c 

Log message:
Fix two bugs.  The first affected tar files with the same
directory listed twice with nothing created inside the directory
in between the two instances of the directory.  The other fixes
extracting symlinks when the -C option is used.  From guenther@
OK krw@



CVS: cvs.openbsd.org: src

2015-02-15 Thread Miod Vallat
CVSROOT:/cvs
Module name:src
Changes by: m...@cvs.openbsd.org2015/02/15 14:34:33

Modified files:
sys/arch/alpha/include: pmap.h 
sys/arch/amd64/include: pmap.h 
sys/arch/arm/include: pmap.h 
sys/arch/hppa/include: pmap.h 
sys/arch/hppa64/include: pmap.h 
sys/arch/i386/include: pmap.h 
sys/arch/m88k/include: pmap.h 
sys/arch/mips64/include: pmap.h 
sys/arch/powerpc/include: pmap.h 
sys/arch/sh/include: pmap.h 
sys/arch/solbourne/include: pmap.h 
sys/arch/sparc/include: pmap.h 
sys/arch/sparc/sparc: pmap.c 
sys/arch/sparc64/sparc64: pmap.c 
sys/arch/vax/vax: pmap.c 
sys/uvm: uvm_map.c uvm_pmap.h 

Log message:
Change pmap_remove_holes() to take a vmspace instead of a map as its argument.

Use this on vax to correctly pick the end of the stack area now that the
stackgap adjustment code will no longer guarantee it is a fixed location.



Re: CVS: cvs.openbsd.org: src

2015-02-15 Thread Henning Brauer
* Stuart Henderson  [2015-02-15 00:32]:
> CVSROOT:  /cvs
> Module name:  src
> Changes by:   st...@cvs.openbsd.org   2015/02/14 16:32:41
> 
> Modified files:
>   sys/net: pf.c 
>   sbin/pfctl : parse.y pfctl_parser.c 
> 
> Log message:
> Rather than using 0xff as a placeholder for "don't check prio", use 0xff to
> mean "prio is 0". This avoids the need for code changes in programs which add
> pf rules (as was done in pfctl but not other programs) to handle the new
> "check prio" functionality. Specifically this unbreaks ftp-proxy.
> 
> Use of #define rather than magic 0xff suggested by benno.
> ok benno "if henning doesnt like it he can change it when he recovers from 
> jet-lag"

nope, perfectly fine. Thanks for handling and sorry for missing this case.

-- 
Henning Brauer, h...@bsws.de, henn...@openbsd.org
BS Web Services GmbH, http://bsws.de, Full-Service ISP
Secure Hosting, Mail and DNS. Virtual & Dedicated Servers, Root to Fully Managed
Henning Brauer Consulting, http://henningbrauer.com/



CVS: cvs.openbsd.org: src

2015-02-15 Thread Ingo Schwarze
CVSROOT:/cvs
Module name:src
Changes by: schwa...@cvs.openbsd.org2015/02/15 10:57:26

Modified files:
share/man/man7 : mdoc.7 

Log message:
Tweak the wording to avoid the possible misunderstanding that .In
could only be used in the SYNOPSIS section.  It is fine anywhere.
Issue noticed by bentley@.



CVS: cvs.openbsd.org: src

2015-02-15 Thread Miod Vallat
CVSROOT:/cvs
Module name:src
Changes by: m...@cvs.openbsd.org2015/02/15 07:35:30

Modified files:
lib/libssl/src/crypto/asn1: asn1_err.c 
lib/libssl/src/crypto/evp: evp_err.c 
lib/libssl/src/crypto/rsa: rsa_err.c 

Log message:
Regen



CVS: cvs.openbsd.org: src

2015-02-15 Thread Joel Sing
CVSROOT:/cvs
Module name:src
Changes by: js...@cvs.openbsd.org   2015/02/15 06:43:32

Modified files:
usr.sbin/httpd : httpd.conf.5 

Log message:
Document the tls protocols option.



CVS: cvs.openbsd.org: src

2015-02-15 Thread Joel Sing
CVSROOT:/cvs
Module name:src
Changes by: js...@cvs.openbsd.org   2015/02/15 06:36:06

Modified files:
lib/libtls : tls_init.3 

Log message:
s/tls_load_keys/tls_load_file/



CVS: cvs.openbsd.org: src

2015-02-15 Thread Joel Sing
CVSROOT:/cvs
Module name:src
Changes by: js...@cvs.openbsd.org   2015/02/15 06:33:14

Modified files:
lib/libtls : Makefile tls_init.3 

Log message:
Document tls_config_parse_protocols() and update documentation for
tls_config_set_protocols().



CVS: cvs.openbsd.org: src

2015-02-15 Thread Ted Unangst
CVSROOT:/cvs
Module name:src
Changes by: t...@cvs.openbsd.org2015/02/15 04:42:30

Modified files:
bin/stty   : stty.c 

Log message:
-a is documented



CVS: cvs.openbsd.org: src

2015-02-15 Thread Stuart Henderson
CVSROOT:/cvs
Module name:src
Changes by: st...@cvs.openbsd.org   2015/02/15 03:52:08

Modified files:
lib/libpcap: pcap-bpf.c 

Log message:
Teach monitor_mode() that the return code for unknown ioctl requests was
changed from EINVAL to ENOTTY in sys/net/if_media.c r1.19; it should just
indicate that monitor mode is not available, rather than return a general
error.  ok dcoppa jca



CVS: cvs.openbsd.org: src

2015-02-15 Thread Stuart Henderson
CVSROOT:/cvs
Module name:src
Changes by: st...@cvs.openbsd.org   2015/02/15 03:40:53

Modified files:
sys/net: pfvar.h 

Log message:
Rather than using 0xff as a placeholder for "don't check prio", use 0xff to
mean "prio is 0". This avoids the need for code changes in programs which add
pf rules (as was done in pfctl but not other programs) to handle the new
"check prio" functionality. Specifically this unbreaks ftp-proxy.

Use of #define rather than magic 0xff suggested by benno.
ok benno "if henning doesnt like it he can change it when he recovers from 
jet-lag"

(this file missed from previous commit, noticed by daniel@)



CVS: cvs.openbsd.org: src

2015-02-15 Thread Marc Espie
CVSROOT:/cvs
Module name:src
Changes by: es...@cvs.openbsd.org   2015/02/15 02:54:21

Modified files:
usr.sbin/pkg_add: fw_update.1 
usr.sbin/pkg_add/OpenBSD: FwUpdate.pm 

Log message:
-i option, so that even pkg_info is not needed.

make the arguments a bit more free-form, so that both radeondrm and
radeondrm-firmware work.



CVS: cvs.openbsd.org: src

2015-02-15 Thread Miod Vallat
CVSROOT:/cvs
Module name:src
Changes by: m...@cvs.openbsd.org2015/02/15 02:46:47

Modified files:
regress/lib/libcrypto/bn/general: bntest.c 

Log message:
Fix various memory leaks by not exiting so abruptly from failed tests.



CVS: cvs.openbsd.org: src

2015-02-15 Thread Miod Vallat
CVSROOT:/cvs
Module name:src
Changes by: m...@cvs.openbsd.org2015/02/15 02:23:23

Modified files:
regress/lib/libcrypto/bf: bftest.c 

Log message:
Remove ancient gcc workaround on mips.



CVS: cvs.openbsd.org: src

2015-02-15 Thread Miod Vallat
CVSROOT:/cvs
Module name:src
Changes by: m...@cvs.openbsd.org2015/02/15 02:17:38

Modified files:
regress/lib/libcrypto/aeswrap: aes_wrap.c 

Log message:
Memory leak. Coverity CID 78865



CVS: cvs.openbsd.org: src

2015-02-15 Thread Miod Vallat
CVSROOT:/cvs
Module name:src
Changes by: m...@cvs.openbsd.org2015/02/15 02:15:41

Modified files:
regress/lib/libcrypto/aead: aeadtest.c 

Log message:
Wrong logic; Coverity CID 78894



CVS: cvs.openbsd.org: src

2015-02-15 Thread Miod Vallat
CVSROOT:/cvs
Module name:src
Changes by: m...@cvs.openbsd.org2015/02/15 01:48:24

Modified files:
lib/libssl/src/crypto/dsa: dsa_gen.c 

Log message:
If we decide to discard the provided seed buffer because its size is not
large enough, do it correctly so that the local seed buffer on the stack
gets properly initialized in the first iteration of the loop.
While there, remove an outdated and bogus comment.
Coverity CID 21785
ok doug@ jsing@



CVS: cvs.openbsd.org: src

2015-02-15 Thread Miod Vallat
CVSROOT:/cvs
Module name:src
Changes by: m...@cvs.openbsd.org2015/02/15 01:45:27

Modified files:
lib/libssl/src/crypto/x509v3: v3_ocsp.c 

Log message:
Check ASN1_OCTET_STRING_new() for failure. Coverity CID 78904
ok doug@



CVS: cvs.openbsd.org: src

2015-02-15 Thread Miod Vallat
CVSROOT:/cvs
Module name:src
Changes by: m...@cvs.openbsd.org2015/02/15 01:44:35

Modified files:
lib/libssl/src/crypto/ec: ec_mult.c 

Log message:
In ec_wNAF_mul(), move the declaration of tmp_wNAF higher in scope, so that
all the function's exit paths can make sure it gets freed. Coverity CID 78861
tweaks & ok doug@ jsing@