svn commit: r315187 - head/lib/libc/stdlib

2017-03-12 Thread Pedro F. Giffuni
Author: pfg Date: Mon Mar 13 04:14:03 2017 New Revision: 315187 URL: https://svnweb.freebsd.org/changeset/base/315187 Log: Minor libc cleanup: let calloc(3) do the multiplication. MFC after:3 days Modified: head/lib/libc/stdlib/getenv.c Modified: head/lib/libc/stdlib/getenv.c

svn commit: r315186 - head/lib/libcam

2017-03-12 Thread Ngie Cooper
Author: ngie Date: Mon Mar 13 02:12:33 2017 New Revision: 315186 URL: https://svnweb.freebsd.org/changeset/base/315186 Log: lib/libcam/cam.3: note that cam_freeccb(3) with ccb == NULL is a no-op This allows me to accurately test this scenario, and for others to rely on the behavior,

Re: svn commit: r315112 - in head: sbin/geom/class/label sys/geom/part

2017-03-12 Thread Ngie Cooper (yaneurabeya)
> On Mar 11, 2017, at 20:15, Marcelo Araujo wrote: > > Author: araujo > Date: Sun Mar 12 04:15:56 2017 > New Revision: 315112 > URL: https://svnweb.freebsd.org/changeset/base/315112 > > Log: > Add the capability to refresh the gpart(8) label without need a reboot. > >

Re: svn commit: r314780 - head/lib/libpam/modules/pam_exec

2017-03-12 Thread Bruce Evans
On Sun, 12 Mar 2017, Ngie Cooper wrote: On Mar 12, 2017, at 11:04, Ian Lepore wrote: ... A quirk of C that's often forgotten is that a function-like macro is only expanded as a macro if the token following the macro name is an open paren. So nitems() is a macro

svn commit: r315181 - head/sbin/setkey

2017-03-12 Thread Ngie Cooper
Author: ngie Date: Mon Mar 13 01:05:25 2017 New Revision: 315181 URL: https://svnweb.freebsd.org/changeset/base/315181 Log: Fix CFLAGS for including netipsec headers #includes The netipsec headers are referenced via netipsec/..., not ./... . Thus, assuming that the netipsec/... is nested

Re: svn commit: r314780 - head/lib/libpam/modules/pam_exec

2017-03-12 Thread Ngie Cooper
> On Mar 12, 2017, at 11:04, Ian Lepore wrote: ... > A quirk of C that's often forgotten is that a function-like macro is > only expanded as a macro if the token following the macro name is an > open paren. So nitems() is a macro invokation and nitems = 0; is just > a

svn commit: r315180 - head/usr.bin/diff

2017-03-12 Thread Baptiste Daroussin
Author: bapt Date: Sun Mar 12 20:19:37 2017 New Revision: 315180 URL: https://svnweb.freebsd.org/changeset/base/315180 Log: Readd codes that creates a tmp file for diffing stdout or devices Modified: head/usr.bin/diff/diffreg.c Modified: head/usr.bin/diff/diffreg.c

svn commit: r315178 - head/sys/dev/evdev

2017-03-12 Thread Oleksandr Tymoshenko
Author: gonzo Date: Sun Mar 12 19:28:44 2017 New Revision: 315178 URL: https://svnweb.freebsd.org/changeset/base/315178 Log: [evdev] Fix Right Alt and Keypad Enter event codes for atkbd(4) and kbdmux(4) drivers Submitted by: Vladimir Kondratiev MFC after:1 week

svn commit: r315177 - head/sys/dev/evdev

2017-03-12 Thread Oleksandr Tymoshenko
Author: gonzo Date: Sun Mar 12 19:27:44 2017 New Revision: 315177 URL: https://svnweb.freebsd.org/changeset/base/315177 Log: [evdev] Fix race condition between client's event queue reading and dropping Submitted by: Vladimir Kondratiev MFC after:1 week

Re: svn commit: r314780 - head/lib/libpam/modules/pam_exec

2017-03-12 Thread Pedro Giffuni
On 3/12/2017 1:04 PM, Ian Lepore wrote: On Sun, 2017-03-12 at 12:30 -0500, Pedro Giffuni wrote: On 3/12/2017 12:14 PM, Lawrence Stewart wrote: Hi Pedro, On 07/03/2017 02:45, Pedro F. Giffuni wrote: Author: pfg Date: Mon Mar 6 15:45:46 2017 New Revision: 314780 URL:

svn commit: r315176 - head/sys/dev/evdev

2017-03-12 Thread Oleksandr Tymoshenko
Author: gonzo Date: Sun Mar 12 19:26:24 2017 New Revision: 315176 URL: https://svnweb.freebsd.org/changeset/base/315176 Log: [evdev] Do not ignore result evdev_register in UI_DEV_CREATE handler Make sure that uinput state field reflects actual state by checking evdev_register result for

svn commit: r315175 - in head/gnu: lib/csu lib/libdialog lib/libgcov lib/libgomp lib/libreadline lib/libregex lib/libssp lib/libssp/libssp_nonshared lib/libstdc++ usr.bin/cc usr.bin/cc/include usr....

2017-03-12 Thread Warner Losh
Author: imp Date: Sun Mar 12 18:59:16 2017 New Revision: 315175 URL: https://svnweb.freebsd.org/changeset/base/315175 Log: Convert gnu to using SRCTOP Prefer SRCTOP over CURDIR/../../contrib, etc. However, retain the "up one level" instances of ../ because they are really relative to

svn commit: r315174 - in head/rescue: librescue rescue

2017-03-12 Thread Warner Losh
Author: imp Date: Sun Mar 12 18:59:09 2017 New Revision: 315174 URL: https://svnweb.freebsd.org/changeset/base/315174 Log: Make rescue use SRCTOP Transition to SRCTOP. Also, standardize of {} instead of () variable deliminators while I'm here. Differential Revision:

svn commit: r315173 - in head/lib: libipsec libpathconv

2017-03-12 Thread Warner Losh
Author: imp Date: Sun Mar 12 18:59:05 2017 New Revision: 315173 URL: https://svnweb.freebsd.org/changeset/base/315173 Log: Fix two CURDIR references in comments that should be SRCTOP references. Differential Revision:https://reviews.freebsd.org/D9932 Sponsored by:

svn commit: r315172 - head/include

2017-03-12 Thread Warner Losh
Author: imp Date: Sun Mar 12 18:59:00 2017 New Revision: 315172 URL: https://svnweb.freebsd.org/changeset/base/315172 Log: Convert include over to SRCTOP Use SRCTOP in place of .CURDIR/.. as appropriate. The hand-crafted relative paths for the "links" option remain, though, since those

svn commit: r315171 - in head/etc: . sendmail

2017-03-12 Thread Warner Losh
Author: imp Date: Sun Mar 12 18:58:55 2017 New Revision: 315171 URL: https://svnweb.freebsd.org/changeset/base/315171 Log: Move /etc/ to SRCTOP Prefer ${SRCTOP}/ to ${.CURDIR}/../ and ${.CURDIR}/../../ as appropriate. Differential Revision: https://reviews.freebsd.org/D9932

svn commit: r315170 - in head/usr.bin: bsdiff/bsdiff bzip2 bzip2recover chkey chpass compile_et cpio ctlstat drill ee elfcopy factor file ftp host ipcrm iscsictl kdump less lessecho lesskey lex lex...

2017-03-12 Thread Warner Losh
Author: imp Date: Sun Mar 12 18:58:44 2017 New Revision: 315170 URL: https://svnweb.freebsd.org/changeset/base/315170 Log: Adopt SRCTOP in usr.bin Prefer ${SRCTOP}/foo over ${.CURDIR}/../../foo and ${SRCTOP}/usr.bin/foo over ${.CURDIR}/../foo for paths in Makefiles. Differential

Re: svn commit: r315164 - head/lib/libpam/modules/pam_exec

2017-03-12 Thread Ngie Cooper
> On Mar 12, 2017, at 10:41, Pedro F. Giffuni wrote: > > Author: pfg > Date: Sun Mar 12 17:41:51 2017 > New Revision: 315164 > URL: https://svnweb.freebsd.org/changeset/base/315164 > > Log: > Revert r314780 > libpam: extra bounds checking through reallocarray(3). > > It

svn commit: r315167 - head/sys/kern

2017-03-12 Thread Ian Lepore
Author: ian Date: Sun Mar 12 18:07:03 2017 New Revision: 315167 URL: https://svnweb.freebsd.org/changeset/base/315167 Log: Change 'Hz' back to 'HZ'... it's referring to the kernel config option named HZ, not being used as an abbreviation of the unit of measure. Modified:

Re: svn commit: r315165 - head/sys/kern

2017-03-12 Thread Ian Lepore
On Sun, 2017-03-12 at 12:45 -0500, Benjamin Kaduk wrote: > On Sun, Mar 12, 2017 at 12:43 PM, Ian Lepore wrote: > > > > > Author: ian > > Date: Sun Mar 12 17:43:45 2017 > > New Revision: 315165 > > URL: https://svnweb.freebsd.org/changeset/base/315165 > > > > Log: > >  

Re: svn commit: r314780 - head/lib/libpam/modules/pam_exec

2017-03-12 Thread Ian Lepore
On Sun, 2017-03-12 at 12:30 -0500, Pedro Giffuni wrote: > > On 3/12/2017 12:14 PM, Lawrence Stewart wrote: > > > > Hi Pedro, > > > > On 07/03/2017 02:45, Pedro F. Giffuni wrote: > > > > > > Author: pfg > > > Date: Mon Mar  6 15:45:46 2017 > > > New Revision: 314780 > > > URL:

svn commit: r315166 - head/sys/dev/acpica

2017-03-12 Thread Warner Losh
Author: imp Date: Sun Mar 12 18:00:02 2017 New Revision: 315166 URL: https://svnweb.freebsd.org/changeset/base/315166 Log: Export the actual LID state via sysctl. This allows more complex policies about when to suspend based on other events happening AND the lid being closed through the use

Re: svn commit: r314780 - head/lib/libpam/modules/pam_exec

2017-03-12 Thread Pedro Giffuni
On 3/12/2017 12:40 PM, Lawrence Stewart wrote: On 13/03/2017 04:30, Pedro Giffuni wrote: On 3/12/2017 12:14 PM, Lawrence Stewart wrote: Hi Pedro, On 07/03/2017 02:45, Pedro F. Giffuni wrote: Author: pfg Date: Mon Mar 6 15:45:46 2017 New Revision: 314780 URL:

Re: svn commit: r315165 - head/sys/kern

2017-03-12 Thread Benjamin Kaduk
On Sun, Mar 12, 2017 at 12:43 PM, Ian Lepore wrote: > Author: ian > Date: Sun Mar 12 17:43:45 2017 > New Revision: 315165 > URL: https://svnweb.freebsd.org/changeset/base/315165 > > Log: > Correct the abbreviations for microseconds (us, not ms), and for Hz (not > HZ). > >

svn commit: r315165 - head/sys/kern

2017-03-12 Thread Ian Lepore
Author: ian Date: Sun Mar 12 17:43:45 2017 New Revision: 315165 URL: https://svnweb.freebsd.org/changeset/base/315165 Log: Correct the abbreviations for microseconds (us, not ms), and for Hz (not HZ). Modified: head/sys/kern/vfs_subr.c Modified: head/sys/kern/vfs_subr.c

svn commit: r315164 - head/lib/libpam/modules/pam_exec

2017-03-12 Thread Pedro F. Giffuni
Author: pfg Date: Sun Mar 12 17:41:51 2017 New Revision: 315164 URL: https://svnweb.freebsd.org/changeset/base/315164 Log: Revert r314780 libpam: extra bounds checking through reallocarray(3). It appears to be causing brokenness when reporting PAM_* environment variables. This requires

Re: svn commit: r314780 - head/lib/libpam/modules/pam_exec

2017-03-12 Thread Lawrence Stewart
On 13/03/2017 04:30, Pedro Giffuni wrote: > > > On 3/12/2017 12:14 PM, Lawrence Stewart wrote: >> Hi Pedro, >> >> On 07/03/2017 02:45, Pedro F. Giffuni wrote: >>> Author: pfg >>> Date: Mon Mar 6 15:45:46 2017 >>> New Revision: 314780 >>> URL: https://svnweb.freebsd.org/changeset/base/314780 >>>

Re: svn commit: r314780 - head/lib/libpam/modules/pam_exec

2017-03-12 Thread Pedro Giffuni
On 3/12/2017 12:14 PM, Lawrence Stewart wrote: Hi Pedro, On 07/03/2017 02:45, Pedro F. Giffuni wrote: Author: pfg Date: Mon Mar 6 15:45:46 2017 New Revision: 314780 URL: https://svnweb.freebsd.org/changeset/base/314780 Log: libpam: extra bounds checking through reallocarray(3).

Re: svn commit: r314780 - head/lib/libpam/modules/pam_exec

2017-03-12 Thread Lawrence Stewart
Hi Pedro, On 07/03/2017 02:45, Pedro F. Giffuni wrote: > Author: pfg > Date: Mon Mar 6 15:45:46 2017 > New Revision: 314780 > URL: https://svnweb.freebsd.org/changeset/base/314780 > > Log: > libpam: extra bounds checking through reallocarray(3). > > Reviewed by:des > MFC after:

svn commit: r315163 - head/sys/dev/isp

2017-03-12 Thread Alexander Motin
Author: mav Date: Sun Mar 12 16:55:18 2017 New Revision: 315163 URL: https://svnweb.freebsd.org/changeset/base/315163 Log: Remove strange config_intrhook_establish() magic. Interrupts are enabled as part of chip reset just during driver attach. Later "enabling" of already enabled

svn commit: r315162 - in head/lib/libc: gen iconv net regex rpc stdio stdlib

2017-03-12 Thread Pedro F. Giffuni
Author: pfg Date: Sun Mar 12 16:03:34 2017 New Revision: 315162 URL: https://svnweb.freebsd.org/changeset/base/315162 Log: libc: provide some bounds-checking through reallocarray(3). reallocarray(3) is a non portable extension that originated in OpenBSD. Given that it is already in

svn commit: r315161 - head/sys/dev/isp

2017-03-12 Thread Alexander Motin
Author: mav Date: Sun Mar 12 15:36:07 2017 New Revision: 315161 URL: https://svnweb.freebsd.org/changeset/base/315161 Log: Try to slight untangle I/O and loop status handling. MFC after:2 weeks Modified: head/sys/dev/isp/isp.c head/sys/dev/isp/isp_freebsd.c

Re: svn commit: r315068 - head/sys/modules/qlxgbe

2017-03-12 Thread Bruce Evans
On Sun, 12 Mar 2017, Andriy Gapon wrote: On 12/03/2017 04:44, Bruce Evans wrote: On Sat, 11 Mar 2017, Andriy Gapon wrote: Log: revert r314862, use of GCC_MS_EXTENSIONS in qlxgbe Commit r278913 made explicit use of GCC_MS_EXTENSIONS unnecessary. Requested by:glebius Does this

svn commit: r315160 - head/sys/dev/isp

2017-03-12 Thread Alexander Motin
Author: mav Date: Sun Mar 12 14:17:57 2017 New Revision: 315160 URL: https://svnweb.freebsd.org/changeset/base/315160 Log: Remove code for unsupported FreeBSD versions. MFC after:2 weeks Modified: head/sys/dev/isp/isp.c head/sys/dev/isp/isp_freebsd.c

Re: svn commit: r308996 - head/lib/libfetch

2017-03-12 Thread Ngie Cooper
> On Mar 12, 2017, at 04:26, Dag-Erling Smørgrav wrote: > > "Ngie Cooper (yaneurabeya)" writes: >> Hi Bjoern, > > No, that was my father :) :D.. >>Shouldn’t this be strrchr instead of strchr since you’re >> scanning for the service from the right side

svn commit: r315159 - head/sys/kern

2017-03-12 Thread Konstantin Belousov
Author: kib Date: Sun Mar 12 13:58:51 2017 New Revision: 315159 URL: https://svnweb.freebsd.org/changeset/base/315159 Log: Avoid reusing p_ksi while it is on queue. When sending SIGCHLD informing reaper that a zombie was reparented to it, we might race with the situation where the

svn commit: r315158 - head/sys/vm

2017-03-12 Thread Konstantin Belousov
Author: kib Date: Sun Mar 12 13:53:13 2017 New Revision: 315158 URL: https://svnweb.freebsd.org/changeset/base/315158 Log: Follow-up to r313690. Fix two missed places where vm_object offset to index calculation should use unsigned shift, to allow handling of full range of unsigned

svn commit: r315157 - head/sys/kern

2017-03-12 Thread Konstantin Belousov
Author: kib Date: Sun Mar 12 13:51:13 2017 New Revision: 315157 URL: https://svnweb.freebsd.org/changeset/base/315157 Log: Accept linkers representation for ELF segments with zero on-disk length. For such segments, GNU bfd linker writes knowingly incorrect value into the the file offset

svn commit: r315156 - head/sys/kern

2017-03-12 Thread Konstantin Belousov
Author: kib Date: Sun Mar 12 13:49:42 2017 New Revision: 315156 URL: https://svnweb.freebsd.org/changeset/base/315156 Log: Style. Sponsored by: The FreeBSD Foundation MFC after:1 week Modified: head/sys/kern/imgact_elf.c Modified: head/sys/kern/imgact_elf.c

svn commit: r315155 - in head/sys: kern sys

2017-03-12 Thread Konstantin Belousov
Author: kib Date: Sun Mar 12 13:48:24 2017 New Revision: 315155 URL: https://svnweb.freebsd.org/changeset/base/315155 Log: Ktracing kevent(2) calls with unusual arguments might leads to an overly large allocation requests. When ktrace-ing io, sys_kevent() allocates memory to copy the

svn commit: r315146 - head/share/man/man4

2017-03-12 Thread Alexander Motin
Author: mav Date: Sun Mar 12 12:38:16 2017 New Revision: 315146 URL: https://svnweb.freebsd.org/changeset/base/315146 Log: Some more polishing. MFC after:13 days Modified: head/share/man/man4/ctl.4 Modified: head/share/man/man4/ctl.4

svn commit: r315143 - head/lib/libfetch

2017-03-12 Thread Dag-Erling Smørgrav
Author: des Date: Sun Mar 12 11:54:35 2017 New Revision: 315143 URL: https://svnweb.freebsd.org/changeset/base/315143 Log: Add a __printflike() that would have caught the bug fixed in r314396. Modified: head/lib/libfetch/common.h Modified: head/lib/libfetch/common.h

Re: svn commit: r308996 - head/lib/libfetch

2017-03-12 Thread Dag-Erling Smørgrav
"Ngie Cooper (yaneurabeya)" writes: > Hi Bjoern, No, that was my father :) > Shouldn’t this be strrchr instead of strchr since you’re > scanning for the service from the right side of the host, > e.g. “foo:service” or “[::1]:service”? This code would match index 3 >

svn commit: r315142 - head/usr.bin/man

2017-03-12 Thread Tijl Coosemans
Author: tijl Date: Sun Mar 12 10:56:19 2017 New Revision: 315142 URL: https://svnweb.freebsd.org/changeset/base/315142 Log: - Remove separate handling of /bin and /usr/bin in manpath. They are no longer a special case. - Prefer PREFIX/share/man over PREFIX/man. - Add

Re: svn commit: r315068 - head/sys/modules/qlxgbe

2017-03-12 Thread Andriy Gapon
On 12/03/2017 04:44, Bruce Evans wrote: > On Sat, 11 Mar 2017, Andriy Gapon wrote: > >> Log: >> revert r314862, use of GCC_MS_EXTENSIONS in qlxgbe >> >> Commit r278913 made explicit use of GCC_MS_EXTENSIONS unnecessary. >> >> Requested by:glebius > > Does this module actually use ms

Re: svn commit: r315141 - head/sys/mips/conf

2017-03-12 Thread Ronald Klop
Hi, Your commit message shows in English what the diff already shows: the 'what changed'. IMHO it would add more value to the commit message to describe the 'why it changed'. What are your ideas about that? Regards, Ronald. On Sun, 12 Mar 2017 08:09:51 +0100, Emmanuel Vadot