On Sun, Feb 05, 2017 at 12:27:19AM +0100, Jeremie Courreges-Anglas wrote:
The colons used in IPv6 addresses conflicts with the proxy port
specification. Do the right thing for -x ::1:8080, [::1] and
[::1]:8080.
With this patch '-x ::1' is still broken. I think we should either
require/enforce
On Mon, Mar 27, 2017 at 07:39:09PM +0200, Sebastian Benoit wrote:
i commited a err(1, "calloc") instead.
What about the other calls to calloc(3); shouldn't we keep their
respective error messages consistent?
Index: pfctl.c
===
RCS file: /cvs/src/sbin/pfctl/pfctl.c,v
retrieving revision 1.338
diff -u -p -r1.338 pfctl.c
--- pfctl.c 26 Jan 2017 08:24:34 - 1.338
+++ pfctl.c 25 Mar 2017 11:37:01 -
@@ -753,7 +753,7 @@
Hey,
blocknonip's description is a tad clearer that way, the rest is mostly
cosmetical but still.
Index: sbin/ifconfig/ifconfig.8
===
RCS file: /cvs/src/sbin/ifconfig/ifconfig.8,v
retrieving revision 1.280
diff -u -p -r1.280
Index: lib/libc/gen/sysctl.3
===
RCS file: /cvs/src/lib/libc/gen/sysctl.3,v
retrieving revision 1.277
diff -u -p -r1.277 sysctl.3
--- lib/libc/gen/sysctl.3 16 Mar 2017 14:57:15 - 1.277
+++ lib/libc/gen/sysctl.3
Now that protocol version 1 was finally dropped in sshd(8), get rid of
this file completely. Our default sshd_config(5) overwrites
AuthorizedKeysFile to ignore it anyway and sshd(8)'s FILES section
doesn't mention it either.
Index: etc/changelist
On Mon, Aug 14, 2017 at 11:21:03AM -0400, Ted Unangst wrote:
> Klemens Nanni wrote:
> > > + case 'z':
> > > + autoaction = AUTO_SUSPEND;
> > > + autolimit = strtonum(optarg, 1, 100, );
> > &g
On Sun, Aug 13, 2017 at 02:13:42PM +0200, Jesper Wallin wrote:
> On Sun, Aug 13, 2017 at 09:52:22AM +0200, Martijn van Duren wrote:
> > I've also been bitten by this a couple of times, but you can also solve
> > this via the sensorsd framework, which is how I've done it.
>
> Yeah, someone on IRC
On Mon, Jul 17, 2017 at 11:57:02AM +0300, Vadim Zhukov wrote:
> > + for _liba in /usr/lib/lib{c,crypto}; do
> > + _libas="$_libas $(ls $_liba.so.+([0-9.]).a | sort -V | tail
> > -1)"
> > done
> > + _libas=${_libas# }
> >
> > # Remount read-write, if
On Mon, Jul 17, 2017 at 02:14:26PM +0300, Vadim Zhukov wrote:
> 2017-07-17 14:03 GMT+03:00 Klemens Nanni <k...@posteo.org>:
> > On Mon, Jul 17, 2017 at 11:57:02AM +0300, Vadim Zhukov wrote:
> >> > + for _liba in /usr/lib/lib{c,crypto}; do
> >> > +
On Sun, Jul 16, 2017 at 09:09:44AM +, Robert Peichaer wrote:
> The rationale to picking the library versions before remounting was
> to keep the time window having rw /usr as small as possible.
> If that's deemed ok, I'm too OK with switching the steps.
Considering the fact that the now
On Sun, Jul 16, 2017 at 12:41:09PM +, Robert Peichaer wrote:
> On Sun, Jul 16, 2017 at 02:28:59PM +0200, Klemens Nanni wrote:
> > On Sun, Jul 16, 2017 at 12:11:55PM +, Robert Peichaer wrote:
> > > On Sun, Jul 16, 2017 at 01:37:56PM +0200, Klemens Nanni wrote:
> > &
On Sun, Jul 16, 2017 at 03:24:19PM +0200, Klemens Nanni wrote:
> + print -- "$RULES" | pfctl -nf -
That should be 'pfctl -ef -' of course.
This removes on level of indent, avoids the ugly RULES="$RULES ..."
repitition and spares a print.
We could do a 'pfctl -ef -' right away but I kept changing and enabling
clearly seperated. Regarding the leading newlines and tabs of the inner
echo: pf perfectly munges those, no need to clear
On Sun, Jul 16, 2017 at 12:11:55PM +, Robert Peichaer wrote:
> On Sun, Jul 16, 2017 at 01:37:56PM +0200, Klemens Nanni wrote:
> > This removes on level of indent, avoids the ugly RULES="$RULES ..."
> > repitition and spares a print.
> >
> > We could do
On Sun, Jul 16, 2017 at 10:26:25AM +, Robert Peichaer wrote:
> On Sun, Jul 16, 2017 at 03:37:15AM +0200, Klemens Nanni wrote:
> > Why looping over all existing archives, picking the latest version of
> > the current archive, skipping it in case it's already in our list of
>
On Sun, Jul 16, 2017 at 07:18:00AM +0200, Theo Buehler wrote:
> On Sun, Jul 16, 2017 at 03:34:07AM +0200, Klemens Nanni wrote:
> > $_l is not used and picking the latest archive versions is of no use
> > if /usr/lib cannot be written to.
> >
> > This patch applies
On Sun, Jul 16, 2017 at 11:55:09AM +, Robert Peichaer wrote:
> To the contrary what my previous answer might indicate, I don't care
> that much either. If you want to explicitely remove the blank, go for
> it.
Most problems I ever encountered while writing shell code were related
to (nested)
The Internal Field Seperator is meant for this so use it instead of
reading and stripping ':' again.
Feedback? Comments?
Index: rc
===
RCS file: /cvs/src/etc/rc,v
retrieving revision 1.508
diff -u -p -r1.508 rc
--- rc 17 Jul 2017
On Mon, Jul 10, 2017 at 07:49:10PM +0200, Alexander Bluhm wrote:
> On Sun, Jul 02, 2017 at 04:29:39AM +0200, Klemens Nanni wrote:
> > No functional change or bugfix but queue(3) is the for a reason. That
> > way the code even is a tad clearer to me (and two lines shorter).
>
Add missing verb and period.
Index: driver/xf86-video-intel/man/intel.man
===
RCS file: /cvs/xenocara/driver/xf86-video-intel/man/intel.man,v
retrieving revision 1.9
diff -u -p -r1.9 intel.man
---
On Mon, Jul 17, 2017 at 10:38:22PM -0400, Bryan Steele wrote:
> This is an upstream Xorg manual.
>
> https://lists.x.org/mailman/listinfo/xorg-devel
Reported upstream, thanks.
Is tech@ the right place for www diffs?
Index: innovations.html
===
RCS file: /cvs/www/innovations.html,v
retrieving revision 1.51
diff -u -p -r1.51 innovations.html
--- innovations.html3 Jul 2017 23:47:21 - 1.51
+++
On Wed, Jun 14, 2017 at 03:00:11AM +0200, Klemens Nanni wrote:
> Installing -current the other day showed a broken list when picking
> the IPv6 default route just like reported on bugs@ five days ago[1].
1: http://marc.info/?l=openbsd-bugs=149704197715791
>
> This behaviour can b
On Sun, Jul 16, 2017 at 08:15:38PM +, Robert Peichaer wrote:
> > + ifconfig lo0 inet6 >/dev/null 2>&1 &&
>
> Please leave the if-then-fi construct intact. This short form is
> mostly used for on-line commands (with only a few exceptions).
OK.
> > + RULES="$RULES"'
>
> What is
On Sun, Jul 16, 2017 at 10:26:25AM +, Robert Peichaer wrote:
> But I'd like to stay strict matching the filenames.
>
> + for _liba in /usr/lib/lib{c,crypto}; do
> + _libas="$_libas $(ls $_liba.so.+([0-9.]).a | sort -V | tail -1)"
> + done
>
> > + _libas=${_libas# }
$_l is not used and picking the latest archive versions is of no use
if /usr/lib cannot be written to.
This patch applies cleanly before my next one but not vice versa.
Feedback? OK?
Index: rc
===
RCS file: /cvs/src/etc/rc,v
Why looping over all existing archives, picking the latest version of
the current archive, skipping it in case it's already in our list of
selected latest versions or adding it otherwise?
The current code runs ls|sort|tail about n * (v - 1) times for n
different libraries and v versions
On Mon, Jul 03, 2017 at 05:36:52PM +0100, Nicholas Marriott wrote:
> Hi
>
> On Thu, Jun 29, 2017 at 09:29:57PM +0200, Klemens Nanni wrote:
> > While reading file(1)'s code in #openbsd-daily mulander noted that the
> > 'name' and 'use' types were missing from magic(5).
>
On Tue, Jul 25, 2017 at 09:43:48PM +0100, Stuart Henderson wrote:
> On 2017/07/25 20:57, Klemens Nanni wrote:
> > I didn't touch the manual's first sentence mentioning file(1)'s version
> > since nicm@ reimplemented file(1) with revision 1.11 on 2015/04/24. He
> > or someone e
5.9 is out, KOI8 isn't supported anymore.
Index: io.c
===
RCS file: /cvs/src/usr.bin/calendar/io.c,v
retrieving revision 1.44
diff -u -p -r1.44 io.c
--- io.c31 Aug 2016 09:38:47 - 1.44
+++ io.c25 Jul 2017
On Tue, Jul 25, 2017 at 11:56:42PM +0200, Theo Buehler wrote:
> On Tue, Jul 25, 2017 at 11:38:59PM +0200, Klemens Nanni wrote:
> > 5.9 is out, KOI8 isn't supported anymore.
>
> It's at least the third time I see this diff. Pretty sure that Jan Stary
> sent the same quite a w
On Mon, Jul 24, 2017 at 06:11:32PM -0400, Ted Unangst wrote:
> Klemens Nanni wrote:
> > usage() never returns, all functions are to be used within this unit
> > only.
> >
> > Since changes are conflicting, I'll wait for this diff first, but I'd
> > like to
On Mon, Jul 24, 2017 at 03:27:05PM -0600, Theo de Raadt wrote:
> this addiction to static is entirely pointless.
Consider it a matter of taste and leave it out, then. I assume you're
fine with __dead, though?
Index: kill.c
===
RCS
usage() never returns, all functions are to be used within this unit
only.
Since changes are conflicting, I'll wait for this diff first, but I'd
like to remove the void casts for fprintf and use getprogname(3) over
__progname as well.
Feedback? Comments?
Index: kill.c
On Mon, Jul 24, 2017 at 04:17:46PM -0600, Theo de Raadt wrote:
> > I know, this sounds silly. Isn't bringing the code up to modern standards
> > part
> > of maintaining it?
>
> I suppose that's my question:
>
> What is it about __dead that makes it part of "modern standards", when it
> isn't
On Mon, Jul 24, 2017 at 08:31:26PM +0200, Klemens Nanni wrote:
> Add missing operators and make the grammar more readable while using
> spaces and tabs consistently.
Missing bits: operands can be more than "any legacy UNIX file name" of
course, correct that as well. One might jus
test's internal getn() makes integers out of strings although boundaries
are checked for long which leads to wrong test results when operands are
greater than INT_MAX but smaller than LONG_MAX:
$ t() { /bin/test "$1" -lt 0 && : overflow; }
$ t 1
+ /bin/test 1 -lt 0
Add missing operators and make the grammar more readable while using
spaces and tabs consistently.
Feedback? Comments?
Index: test.c
===
RCS file: /cvs/src/bin/test/test.c,v
retrieving revision 1.17
diff -u -p -r1.17 test.c
---
As done in other places already, replace __progname with getprogname(3).
Feedback? Comments?
Index: test.c
===
RCS file: /cvs/src/bin/test/test.c,v
retrieving revision 1.17
diff -u -p -r1.17 test.c
--- test.c 21 Jan 2017
On Thu, Jul 27, 2017 at 10:31:51AM +0100, Larry Hynes wrote:
> $ env | grep LESS
> LESSHISTFILE=-
> LESS="-i -M -R -g -c"
> LESSCHARSET=utf-8
>
> $ unset LESS
> $ unset LESSCHARSET
> $ unset LESSHISTFILE
>
> $ LESS="-g"
> $ echo 'foo\nbar\nfoo\nbar\nfoo\nbar' | less
>
> While in less, '/' to
On Thu, Jul 27, 2017 at 02:03:05PM +0200, Jeremie Courreges-Anglas wrote:
> On Thu, Jul 27 2017, Klemens Nanni <k...@posteo.org> wrote:
> > Only main() calls pr_args() in L330 with ep->kp as argument which in
> > turn is set in L257 or L266 for every utmp entry. kp is
Only main() calls pr_args() in L330 with ep->kp as argument which in
turn is set in L257 or L266 for every utmp entry. kp is checked against
NULL already in L229.
Even if kp was somehow NULL chances are high we'd fail before pr_args()
was called anyway since L244, L256 and L265 would then cause a
While reading file(1)'s code in #openbsd-daily mulander noted that the
'name' and 'use' types were missing from magic(5).
I'm not entirely sure yet whether this is complete, so here's what I
did:
magic(5) provided by devel/magic documents version 5.31 while base's
magic(5) is at 4.24. Here are
On Thu, Jun 29, 2017 at 09:50:25PM -0700, Philip Guenther wrote:
On Thu, Jun 22, 2017 at 7:17 PM, Klemens Nanni <k...@posteo.org> wrote:
So I just wrapped my head around vfs(9) with regard to making ktrace
following symlinks again, however I don't quite understand what problems
may occu
No need for multiple echos or xargs (wich runs cat only once anyway)
here. The {post-,}magic files stay unchanged.
In magic target don't specify dependencies twice.
OK?
Index: Makefile
===
RCS file:
On Sun, Jul 02, 2017 at 01:09:19AM +0100, Ricardo Mestre wrote:
You're blindly replacing strtoul with strtonum without taking the base 0
into account.
I'd recommend you take a look at strtoul's manpage first and check what
using base 0 means (strtonum always use base 10).
Don't worry, I can
No functional change or bugfix but queue(3) is the for a reason. That
way the code even is a tad clearer to me (and two lines shorter).
Feedback/OK?
Index: tee.c
===
RCS file: /cvs/src/usr.bin/tee/tee.c,v
retrieving revision 1.11
On Fri, Jun 09, 2017 at 08:33:16PM +0200, Klemens Nanni wrote:
No need for temporary variables either, strtonum guarantees through
maxval that its return value won't overflow when casted.
Index: brconfig.c
===
RCS file: /cvs/src
On Mon, Jul 03, 2017 at 10:47:31PM +, Robert Peichaer wrote:
Dokument explicitely possible outputs and tweak the sed expressions
to remove the superfluous whitespaces. I guess that does the trick.
Index: install.sub
===
RCS
Remove duplicate full stop and add space after function name.
Feedback/OK?
Index: install.sub
===
RCS file: /cvs/src/distrib/miniroot/install.sub,v
retrieving revision 1.1019
diff -u -p -r1.1019 install.sub
--- install.sub 2 Jul
On Mon, Jul 03, 2017 at 05:36:52PM +0100, Nicholas Marriott wrote:
Hi
On Thu, Jun 29, 2017 at 09:29:57PM +0200, Klemens Nanni wrote:
[...]
What about the current version being 4.21? We're clearly ahead of this,
it seems magic(5) wasn't updated when nicm@ reimplemented things.
This patch
Purely cosmetic/style(9) fixes: Remove unneeded indent in prototypes,
add space after return keyword.
Feedback/OK?
Index: ping.c
===
RCS file: /cvs/src/sbin/ping/ping.c,v
retrieving revision 1.220
diff -u -p -r1.220 ping.c
---
On Mon, Jul 03, 2017 at 12:45:32AM +0200, Klemens Nanni wrote:
Thanks for looking into it.
On Sun, Jul 02, 2017 at 04:32:43PM +, Robert Peichaer wrote:
ieee80211_scan()
- Extract the needed information (nwid, bssid) using a very specific
sed expression. Any line, not matching this expr
On Tue, Jul 04, 2017 at 03:58:13PM +, Florian Obser wrote:
I like most (all? of the correct ones), can you have another go
at trying to fix the pointed out changes in behaviour?
Thanks natano for pointing out these stupid mistakes.
Unify option checking, check for F_RROUTE only if
On Tue, Jul 04, 2017 at 04:00:43PM +, Florian Obser wrote:
yeah, this is arse backwards, I'm willing to commit the oposite though,
i.e. get rid of the void casts for printf
Casts removed, cosecutive calls merged where suitable.
Feedback/OK?
Index: ping.c
Thanks for looking into it.
On Sun, Jul 02, 2017 at 04:32:43PM +, Robert Peichaer wrote:
ieee80211_scan()
- Extract the needed information (nwid, bssid) using a very specific
sed expression. Any line, not matching this expr is ignored.
- Remove leading and trailing double-quotes in
A simple command list suffices as this is just about redirecting all
output at once.
Feedback/OK?
Index: install.sub
===
RCS file: /cvs/src/distrib/miniroot/install.sub,v
retrieving revision 1.1016
diff -u -p -r1.1016 install.sub
On Sat, Jul 01, 2017 at 09:35:09PM -0700, Philip Guenther wrote:
On Sat, Jul 1, 2017 at 9:27 PM, Klemens Nanni <k...@posteo.org> wrote:
A simple command list suffices as this is just about redirecting all
output at once.
Feedback/OK?
...
+ {dmesg; cat $CGI_INFO /*.conf; sysctl;
On Sat, Jul 01, 2017 at 11:01:00PM -0600, Theo de Raadt wrote:
Don't understand your point. What is this fixing?
Nothing gets "fixed", it just avoids unnecessary forking and makes clear
that the inner commands do not require a subshell - there are no side
effects on the environment so a
On Sat, Jul 01, 2017 at 10:13:45PM -0700, Philip Guenther wrote:
I've lost track of how many times at work and in OpenBSD I've tested a
diff, made a final modification and then committed...a change broken
by that final modification. Once you've tested a diff, either don't
make a "this is easy!"
On Sat, Jul 01, 2017 at 11:17:55PM -0600, Theo de Raadt wrote:
On Sat, Jul 01, 2017 at 11:01:00PM -0600, Theo de Raadt wrote:
>Don't understand your point. What is this fixing?
Nothing gets "fixed", it just avoids unnecessary forking and makes clear
that the inner commands do not require a
strmode(3) is void and thus never returns anything.
Feedback/OK?
Index: strmode.3
===
RCS file: /cvs/src/lib/libc/string/strmode.3,v
retrieving revision 1.16
diff -u -p -r1.16 strmode.3
--- strmode.3 5 Jun 2013 03:39:23 -
No need for buffers t0, t1 here. This way we only have to step/move the
current position instead of the total bytes left as well as both source
and destination position.
Always swap an even number of bytes by clearing the length's last bit
and never write past it.
The , operator can be used
On Wed, Jul 05, 2017 at 05:27:18PM +0200, Ingo Schwarze wrote:
Hi Klemens,
Klemens Nanni wrote on Wed, Jul 05, 2017 at 05:02:05PM +0200:
No need for buffers t0, t1 here.
Your patch changes behaviour in some cases where the buffers do
overlap. For example, if src == dst, right now, the code
On Wed, Jul 05, 2017 at 09:47:11AM -0600, Theo de Raadt wrote:
> > So I'd say for cases like src == dst we don't have to guarantee that
> > bytes are swapped.
>
> and you've audited all the callers to this function?
>
> > Agreed, I haven't checked for bad/dangerous usage in existing code for
> >
On Mon, Jun 26, 2017 at 08:26:03PM -0500, Edgar Pettijohn wrote:
I found the following conflict between dhcp-options(5) and dhcpd.conf(5).
From dhcpd.conf:
As you can see in Example 2, it's legal to specify host addresses in
parameters as hostnames rather than as numeric IP addresses. If
On Thu, Jun 29, 2017 at 11:33:36PM -0700, Philip Guenther wrote:
On Thu, Jun 29, 2017 at 10:42 PM, Klemens Nanni <k...@posteo.org> wrote:
On Thu, Jun 29, 2017 at 09:50:25PM -0700, Philip Guenther wrote:
On Thu, Jun 22, 2017 at 7:17 PM, Klemens Nanni <k...@posteo.org> wrote:
Index: guide.html
===
RCS file: /cvs/www/faq/ports/guide.html,v
retrieving revision 1.73
diff -u -p -r1.73 guide.html
--- guide.html 8 Aug 2017 15:48:56 - 1.73
+++ guide.html 9 Aug 2017 00:13:22 -
@@ -808,7 +808,7 @@
Index: faq/ports/guide.html
===
RCS file: /cvs/www/faq/ports/guide.html,v
retrieving revision 1.71
diff -u -p -r1.71 guide.html
--- faq/ports/guide.html26 Jun 2017 17:18:58 - 1.71
+++ faq/ports/guide.html7 Aug
This way it's feels much more natural to say.
Feedback?
Index: guide.html
===
RCS file: /cvs/www/faq/ports/guide.html,v
retrieving revision 1.72
diff -u -p -r1.72 guide.html
--- guide.html 7 Aug 2017 22:27:51 - 1.72
+++
Found while looking for the core dump bits.
Feedback?
Index: README
===
RCS file: /cvs/xenocara/README,v
retrieving revision 1.39
diff -u -p -r1.39 README
--- README 24 Feb 2017 03:07:03 - 1.39
+++ README 18 Aug
On Fri, Jun 02, 2017 at 05:07:42PM +0200, Anton Lindqvist wrote:
Custom completions in ksh is currently limited to commands that does not
contain hyphens since such a character cannot be part of an identifier.
We could cheat and replace hyphens with underscores upon performing
completions.
The
On Tue, Jun 20, 2017 at 09:10:17PM -0400, Ted Unangst wrote:
Klemens Nanni wrote:
I wanted to quickly debug some program without actually dumping to disk
by using FIFOs, however ktrace(2) wouldn't accept anything but regular
files.
Are there any pitfalls or limitations I am currently not aware
I wanted to quickly debug some program without actually dumping to disk
by using FIFOs, however ktrace(2) wouldn't accept anything but regular
files.
Are there any pitfalls or limitations I am currently not aware of that
justify this strict behaviour?
$ ln -s some.file link
$
So I just wrapped my head around vfs(9) with regard to making ktrace
following symlinks again, however I don't quite understand what problems
may occur when doing so. May anyone enlighten me on this?
This feature was explicitly disabled 17 years ago[1] appearently because
FreeBSD did so[2] but I
NIFS is checked inside start_cgiinfo() already, this nicely aligns with
the rest of the list in do_install().
Index: install.sub
===
RCS file: /cvs/src/distrib/miniroot/install.sub,v
retrieving revision 1.1015
diff -u -p -r1.1015
===
RCS file: /cvs/src/distrib/miniroot/install.sub,v
retrieving revision 1.1014
diff -u -p -u -r1.1014 install.sub
--- distrib/miniroot/install.sub3 Jun 2017 22:27:41 - 1.1014
+++ distrib/miniroot/install.sub
On Sun, Jun 25, 2017 at 10:47:08PM -0600, Theo de Raadt wrote:
:-) Speaking of signed integers, does it really need to be signed?
Perhaps not. Anyone know for sure?
Of course this number should probably exclude 0 in it's range.
Probably [0, 3] completely as those are reserved for
On Sun, Jun 25, 2017 at 11:21:50PM -0600, Theo de Raadt wrote:
On Sun, Jun 25, 2017 at 10:47:08PM -0600, Theo de Raadt wrote:
>> :-) Speaking of signed integers, does it really need to be signed?
>
>Perhaps not. Anyone know for sure?
>
>Of course this number should probably exclude 0 in it's
Besides fixing a tautological 'v < 0' and using more descriptive/less
errorprone sizeof(target) this patch unifies strtoul(3) handling both
logically and cosmetically.
Index: brconfig.c
===
RCS file:
-exec has been around since 2012 is cleaner and shorter, I don't see why
xargs(1) should be used for such simple cases. Besides that don't walk
the tree twice.
Index: include/Makefile
===
RCS file: /cvs/src/include/Makefile,v
=X actually works quite fine despite the mentioned condition:
$ touch f; mkdir d
$ chmod =X f d
$ ls -ld f d
d--x--x--x 2 kle wheel 512 Jun 3 15:11 d/
-- 1 kle wheel0 Jun 3 15:11 f
One of the given examples has been updated to reflect
Bump: Feeback? OK?
On Mon, Apr 17, 2017 at 09:28:29PM +0200, Klemens Nanni wrote:
Now that protocol version 1 was finally dropped in sshd(8), get rid of
this file completely. Our default sshd_config(5) overwrites
AuthorizedKeysFile to ignore it anyway and sshd(8)'s FILES section
doesn't mention
On Wed, Jun 07, 2017 at 01:17:10PM -0400, Ted Unangst wrote:
Klemens Nanni wrote:
Besides fixing a tautological 'v < 0' and using more descriptive/less
errorprone sizeof(target) this patch unifies strtoul(3) handling both
logically and cosmetically.
A great deal of these look like exac
This fixes the primitive parsing route of settimeslot() to allow any
possible list of slots and/or ranges, see the update manual section.
The old code would happily mask "1,2-" into 0b11, settimeslot() now fails
on such broken ranges and also checks for inconsistencies like "4-1",
etc.
With this patch, v[46]_info() both output exactly what their description
says.
As of now, these functions are only used through
set -- $(v4_info $_if)
which gracefully handles any constellation of whitespaces in the output
just fine. However future usage might change (or not) and being
On Wed, Jun 14, 2017 at 03:00:11AM +0200, Klemens Nanni wrote:
Installing -current the other day showed a broken list when picking
the IPv6 default route just like reported on bugs@ five days ago[1].
Missed the link.
1: http://marc.info/?l=openbsd-bugs=149704197715791
Installing -current the other day showed a broken list when picking
the IPv6 default route just like reported on bugs@ five days ago[1].
This behaviour can be reproduced manually running the code from
v6_defroute():
$ # source/define bsort()
$ _if=trunk0
$ bsort $(ping6
On Wed, Jun 14, 2017 at 07:54:48PM +0200, Jérôme FRGACIC wrote:
I recently realized that the word "rdonly" is not an alias for "ro" in
the fstab file, which is inconsistent with the mount(1) command. I
would suggest to add support for it with this small patch.
fstab(5) has always used ro/rw
Instead of ignoring SSIDs containing whitespaces, slightly adjust the
commands to take everything in between 'nwid ' and ' chan' as SSID; if
it has double quotes at start *and* end, simply remove those.
This enables users to select networks such as "Unitymedia WifiSpot"
"FRITZ!Box 7490" for
No need for quoting $_nwid within [[ ... ]] since field splitting is not
applied.
Index: install.sub
===
RCS file: /cvs/src/distrib/miniroot/install.sub,v
retrieving revision 1.1014
diff -u -p -r1.1014 install.sub
--- install.sub 3
'arp -[d]a' do not require hostname as told in the manual page.
A single fprintf() is both shorter and cleaner.
Index: arp.8
===
RCS file: /cvs/src/usr.sbin/arp/arp.8,v
retrieving revision 1.39
diff -u -p -r1.39 arp.8
--- arp.8
On Tue, Jun 13, 2017 at 10:45:57AM -0600, Theo de Raadt wrote:
Sorry, but that type of diff is a no-go. You've made a large variety
of different decisions on your own and mixed them up with ones which
are personal taste, and then touched so many lines of the code that
future study of historical
Do not clear the unneeded dst[46] structure.
Index: ping.c
===
RCS file: /cvs/src/sbin/ping/ping.c,v
retrieving revision 1.218
diff -u -p -r1.218 ping.c
--- ping.c 22 Feb 2017 13:43:35 - 1.218
+++ ping.c 13 Jun
printf's return value is ignored allmost all the times so do the same
for the rest of them as well for consistency.
Subsequent calls got merged where I think is appropiate and readability
improves.
The usage format string now looks like it's actual put.
Index: ping.c
The intent here is to get the highest multiple of four smaller or equal
than i + 3. Instead of relying on integer division to get rid of the
remainder just to "undo" everything, simply clear the lowest two bits
(0b11 = 3) leaving multiples of four.
Index: ping.c
Unify option checking and simply logic.
F_HDRINCL and F_ROUTE are mutually exclusive, thus check the latter only
if the former one is not set.
Index: ping.c
===
RCS file: /cvs/src/sbin/ping/ping.c,v
retrieving revision 1.218
diff
No need for temporary variables either, strtonum guarantees through
maxval that its return value won't overflow when casted.
Index: brconfig.c
===
RCS file: /cvs/src/sbin/ifconfig/brconfig.c,v
retrieving revision 1.15
diff -u -p
Ignore return status of all printf calls consistently, merge
subsequent ones where appropiate, do not memset unneeded dst structure,
simplify/unify option checks, use err not perror, break lines at 80
chars, remove unnecessary parentheses, avoid unobvious integer divsion,
make usage format string
1 - 100 of 1834 matches
Mail list logo
