CVS: cvs.openbsd.org: src

2017-05-03 Thread Damien Miller 
CVSROOT:/cvs
Module name:src
Changes by: d...@cvs.openbsd.org2017/05/03 19:33:21

Modified files:
usr.bin/ssh: cipher.c cipher.h 

Log message:
another tentacle: cipher_set_key_string() was only ever used for SSHv1

CVS: cvs.openbsd.org: src

2017-05-03 Thread Mark Kettenis 
CVSROOT:/cvs
Module name:src
Changes by: kette...@cvs.openbsd.org2017/05/03 16:35:49

Modified files:
sys/arch/arm64/arm64: machdep.c 

Log message:
Bring boot() in line with our other architectures.

ok mpi@, deraadt@

CVS: cvs.openbsd.org: src

2017-05-03 Thread Christian Weisgerber 
CVSROOT:/cvs
Module name:src
Changes by: na...@cvs.openbsd.org   2017/05/03 15:49:18

Modified files:
usr.bin/ssh: scp.1 sftp.1 ssh.1 ssh_config.5 

Log message:
restore mistakenly deleted description of the ConnectionAttempts option
ok markus@

CVS: cvs.openbsd.org: src

2017-05-03 Thread Christian Weisgerber 
CVSROOT:/cvs
Module name:src
Changes by: na...@cvs.openbsd.org   2017/05/03 15:08:09

Modified files:
usr.bin/ssh: clientloop.c kex.h opacket.h packet.c packet.h 
 pathnames.h ssh.h ssh_config.5 sshkey.h 

Log message:
remove miscellaneous SSH1 leftovers; ok markus@

CVS: cvs.openbsd.org: src

2017-05-03 Thread Mike Belopuhov 
CVSROOT:/cvs
Module name:src
Changes by: mi...@cvs.openbsd.org   2017/05/03 14:55:29

Modified files:
sys/net: ifq.c ifq.h 

Log message:
Provide a function to dispose of a list of mbufs on dequeue

ifq_mfreeml() is similar to the ifq_mfreem(), but takes an mbuf list
as an argument.  This also lets these functions subtract the number
of packets to be disposed of from the ifq length.

OK dlg

CVS: cvs.openbsd.org: src

2017-05-03 Thread Todd C . Miller 
CVSROOT:/cvs
Module name:src
Changes by: mill...@cvs.openbsd.org 2017/05/03 13:58:12

Modified files:
sys/netinet: tcp_input.c 

Log message:
Back out rev 1.185 (which made the code match the comment) and
adjust the comment to match reality (or at least rfc7323) instead.
This brings us back in line with the behavior of Net and Free.
>From Lauri Tirkkonen.  OK bluhm@

CVS: cvs.openbsd.org: src

2017-05-03 Thread Mike Belopuhov 
CVSROOT:/cvs
Module name:src
Changes by: mi...@cvs.openbsd.org   2017/05/03 11:51:57

Modified files:
sys/sys: mbuf.h 

Log message:
Provide a signed 64 bit integer timestamp in the mbuf packet header

The precision of the timestamp is not fixed yet, but there's a strong
argument to measure it in nanoseconds.

With suggestions from kettenis, dlg, miod and deraadt.
OK deraadt@, sthen@

CVS: cvs.openbsd.org: src

2017-05-03 Thread Antoine Jacoutot 
CVSROOT:/cvs
Module name:src
Changes by: ajacou...@cvs.openbsd.org   2017/05/03 11:23:34

Modified files:
usr.sbin/syspatch: syspatch.sh 

Log message:
Output explicit error messages for:
- trying to install files mounted on a remote FS (diskless clients etc.)
- read-only FS
- not enough space on FS

These were all properly catched before (syspatch would refuse to do anything)
but the error message was a bit cryptic.

Re: CVS: cvs.openbsd.org: src

2017-05-03 Thread Stefan Sperling 
On Wed, May 03, 2017 at 04:54:26PM +0200, Stefan Sperling wrote:
> On Wed, May 03, 2017 at 08:53:12AM -0600, Stefan Sperling wrote:
> > CVSROOT:/cvs
> > Module name:src
> > Changes by: s...@cvs.openbsd.org2017/05/03 08:53:12
> > 
> > Modified files:
> > sys/dev/pci: if_iwmreg.h 
> > 
> > Log message:
> > Fix a regression in iwm(4) which made 3165 devices unusable.
> > 
> > When MIMO support was added several entries in the rate table index enum
> > got a wrong value. On most devices the firmware ended up using a Tx rate
> > different from the rate net80211 asked for (which is bad, but not fatal).
> > But on 3165 devices which do not support MIMO the firmware rightly raised
> > a fatal error whenever the driver mistakenly asked for a MIMO Tx rate.
> > 
> > Reported by Georgios Pediaditis, Steve Throckmorton, and Kai Wirt on misc@
> > Thanks to benno@ for providing 3165 hardware I could use to debug this.
> 
> and ok phessler@

and mpi@

CVS: cvs.openbsd.org: src

2017-05-03 Thread Reyk Floeter 
CVSROOT:/cvs
Module name:src
Changes by: r...@cvs.openbsd.org2017/05/03 09:49:10

Modified files:
lib/libutil: imsg_init.3 

Log message:
spacing (in EXAMPLES code)

CVS: cvs.openbsd.org: src

2017-05-03 Thread Gerhard Roth 
CVSROOT:/cvs
Module name:src
Changes by: gerh...@cvs.openbsd.org 2017/05/03 09:32:31

Modified files:
sys/dev/usb: if_umb.c 

Log message:
Byte order of IP addresses was broken on big endian machines.
MBIM already uses network byte order for IP addresses, so just
use them as they are.

ok stsp@, deraadt@

CVS: cvs.openbsd.org: www

2017-05-03 Thread Juan Francisco Cantero Hurtado 
CVSROOT:/cvs
Module name:www
Changes by: juan...@cvs.openbsd.org 2017/05/03 09:29:36

Modified files:
.  : want.html donations.html 

Log message:
Arnaud Assad sent me a X61s (and the 4.4 BSD book).

Re: CVS: cvs.openbsd.org: src

2017-05-03 Thread Stefan Sperling 
On Wed, May 03, 2017 at 08:53:12AM -0600, Stefan Sperling wrote:
> CVSROOT:  /cvs
> Module name:  src
> Changes by:   s...@cvs.openbsd.org2017/05/03 08:53:12
> 
> Modified files:
>   sys/dev/pci: if_iwmreg.h 
> 
> Log message:
> Fix a regression in iwm(4) which made 3165 devices unusable.
> 
> When MIMO support was added several entries in the rate table index enum
> got a wrong value. On most devices the firmware ended up using a Tx rate
> different from the rate net80211 asked for (which is bad, but not fatal).
> But on 3165 devices which do not support MIMO the firmware rightly raised
> a fatal error whenever the driver mistakenly asked for a MIMO Tx rate.
> 
> Reported by Georgios Pediaditis, Steve Throckmorton, and Kai Wirt on misc@
> Thanks to benno@ for providing 3165 hardware I could use to debug this.

and ok phessler@

CVS: cvs.openbsd.org: src

2017-05-03 Thread Stefan Sperling 
CVSROOT:/cvs
Module name:src
Changes by: s...@cvs.openbsd.org2017/05/03 08:53:12

Modified files:
sys/dev/pci: if_iwmreg.h 

Log message:
Fix a regression in iwm(4) which made 3165 devices unusable.

When MIMO support was added several entries in the rate table index enum
got a wrong value. On most devices the firmware ended up using a Tx rate
different from the rate net80211 asked for (which is bad, but not fatal).
But on 3165 devices which do not support MIMO the firmware rightly raised
a fatal error whenever the driver mistakenly asked for a MIMO Tx rate.

Reported by Georgios Pediaditis, Steve Throckmorton, and Kai Wirt on misc@
Thanks to benno@ for providing 3165 hardware I could use to debug this.

CVS: cvs.openbsd.org: src

2017-05-03 Thread Mike Belopuhov 
CVSROOT:/cvs
Module name:src
Changes by: mi...@cvs.openbsd.org   2017/05/03 08:01:29

Modified files:
usr.bin/systat : pftop.c 

Log message:
Remove unused PRIO column from the queueing display;  ok sthen@

CVS: cvs.openbsd.org: src

2017-05-03 Thread David Gwynne 
CVSROOT:/cvs
Module name:src
Changes by: d...@cvs.openbsd.org2017/05/03 06:27:00

Modified files:
sys/arch/sparc64/sparc64: pmap.c 

Log message:
explicitly initialise the mutex in the kernel map.

previously it was zeroed memory from a global, which kind of looks
like a valid mutex, but we shouldnt rely on that.

ok kettenis@

CVS: cvs.openbsd.org: src

2017-05-03 Thread Antoine Jacoutot 
CVSROOT:/cvs
Module name:src
Changes by: ajacou...@cvs.openbsd.org   2017/05/03 06:26:52

Modified files:
usr.sbin/syspatch: syspatch.sh 

Log message:
- Don't make an empty syspatch/ repo an error; having no available patch
is perfectly legit.
- Abort immediately if we cannot reach our mirror server listed in
installurl.
- Return a proper error code when running without arguments and we cannot
list the available patches.

CVS: cvs.openbsd.org: src

2017-05-03 Thread Todd C . Miller 
CVSROOT:/cvs
Module name:src
Changes by: mill...@cvs.openbsd.org 2017/05/03 05:59:25

Modified files:
usr.bin/less   : tags.c 

Log message:
While freeing tag entries, make sure to free the copied strings.
>From Anton Lindqvist.  OK tobias@ nicm@

CVS: cvs.openbsd.org: src

2017-05-03 Thread Gleydson Soares 
CVSROOT:/cvs
Module name:src
Changes by: gsoa...@cvs.openbsd.org 2017/05/03 05:55:36

Modified files:
etc: changelist 
etc/mtree  : special 

Log message:
add vm.conf to changelist and mtree/special
OK reyk mlarkin

CVS: cvs.openbsd.org: src

2017-05-03 Thread Jason McIntyre 
CVSROOT:/cvs
Module name:src
Changes by: j...@cvs.openbsd.org2017/05/03 04:01:44

Modified files:
usr.bin/ssh: ssh_config 

Log message:
more protocol 1 bits removed; ok djm

CVS: cvs.openbsd.org: src

2017-05-03 Thread Ricardo Mestre 
CVSROOT:/cvs
Module name:src
Changes by: mes...@cvs.openbsd.org  2017/05/03 03:51:39

Modified files:
sbin/init  : init.c 
usr.bin/encrypt: encrypt.c 
usr.bin/lock   : lock.c 
usr.bin/skey   : skey.c 
usr.bin/x99token: x99token.c 
usr.sbin/tokeninit: tokeninit.c 

Log message:
Use the safe idiom of cleaning sensitive data from memory with explicit_bzero,
instead of relying on other methods, after readpassphrase. Some programs on
this diff won't benefit that much since it happens near the terminal path, but
someone might copy the unsafe idiom to another program and place it where it
may leak sensitive data.

Discussed aeons ago with tb@, OK deraadt@ and beck@

CVS: cvs.openbsd.org: src

2017-05-03 Thread Rafael Zalamena 
CVSROOT:/cvs
Module name:src
Changes by: rzalam...@cvs.openbsd.org   2017/05/03 02:35:55

Modified files:
sys/netinet6   : ip6_output.c 

Log message:
Change the ip6_setmoptions() function to receive the rdomain as a new
parameter. This makes the ip6_setmoptions() function look more like the
ipv4 version and fixes a problem with IPV6_JOIN_GROUP when no interface
is specified.

ok bluhm@

CVS: cvs.openbsd.org: src

2017-05-03 Thread Reyk Floeter 
CVSROOT:/cvs
Module name:src
Changes by: r...@cvs.openbsd.org2017/05/03 02:21:57

Modified files:
usr.sbin/vmd   : parse.y 

Log message:
Sort parser tokens, no functional change

CVS: cvs.openbsd.org: src

2017-05-03 Thread Alexandre Ratchov 
CVSROOT:/cvs
Module name:src
Changes by: ratc...@cvs.openbsd.org 2017/05/03 00:58:11

Modified files:
sys/dev/usb: uaudio.c 

Log message:
Remove unused (and wrong) uaudio_drain() call. From Michael W.
Bombardieri" .

CVS: cvs.openbsd.org: src

2017-05-03 Thread Alexandre Ratchov 
CVSROOT:/cvs
Module name:src
Changes by: ratc...@cvs.openbsd.org 2017/05/03 00:56:54

Modified files:
sys/dev: audio.c 

Log message:
style improvement from Michael W. Bombardieri

CVS: cvs.openbsd.org: src

2017-05-03 Thread Jonathan Gray 
CVSROOT:/cvs
Module name:src
Changes by: j...@cvs.openbsd.org2017/05/03 00:45:48

Modified files:
sys/dev/pci: Tag: OPENBSD_6_1 if_ix.c 

Log message:
MFC

Since rev 1.1 ix has attempted to require a 64 bit BAR, the test for
this was wrong and was corrected at the end of last year in rev 1.140.
Before then a 64 bit BAR was not enforced as the test was wrong.

It turns out there exist 82598 parts which have a 32 bit BAR so change
the test to only require a memory BAR and not a 64 bit memory BAR.

Problem reported by Robert Blacquiere. ok mikeb@

CVS: cvs.openbsd.org: src

2017-05-03 Thread Jason McIntyre 
CVSROOT:/cvs
Module name:src
Changes by: j...@cvs.openbsd.org2017/05/03 00:32:02

Modified files:
usr.bin/ssh: sftp.1 ssh-add.1 ssh-keygen.1 ssh.1 

Log message:
more protocol 1 stuff to go; ok djm

CVS: cvs.openbsd.org: www

2017-05-03 Thread Mike Larkin 
CVSROOT:/cvs
Module name:www
Changes by: mlar...@cvs.openbsd.org 2017/05/03 00:15:51

Modified files:
.  : errata61.html 

Log message:
clarify the recent vmm errata was for amd64, not "All Architectures"