- Original Message -
From: "Jason McIntyre"
To: "tech"
Sent: Sunday, September 6, 2015 3:15:34 PM
Subject: Re: remove redundancy in tame(2) man page
On Sun, Sep 06, 2015 at 02:30:43PM -0400, Rob Pierce wrote:
> fstat(2) mentioned twice under wpath.
This seems to be the result of the programmer misunderstanding strsep()
(unless I'm misunderstanding something). The return value is the
supplied string pointer. Because strsep() is called in a while loop,
each iteration's hn is the last iteration's inputstring. We therefore
have to keep a
Stefan Fritsch (2015-09-06 11:44 +0200):
> the diff below is necessary to make suspend/resume work when x2apic is
> enabled (i.e. on qemu/kvm/...). While I don't expect problems, it would be
> nice if I could get some reports that this doesn't break suspend/resume on
> real machines, especially
The first is a pretty trivial change for clarity and correctness.
The second probably isn't the best way to solve the problem long-term,
but prevents a div-by-zero two lines later. I may have been affected by
it when fscking a /dev/zero'd partition.
Found with the Clang static analyzer (and the
Found with the Clang static analyzer.
Index: c_ksh.c
===
RCS file: /cvs/src/bin/ksh/c_ksh.c,v
retrieving revision 1.35
diff -u -p -r1.35 c_ksh.c
--- c_ksh.c 1 Sep 2015 13:12:31 - 1.35
+++ c_ksh.c 7 Sep 2015
Hi,
tcpdump(8) prints radius message dumps incorrectly by skipping leading zeros
while printing hex values. This patch fixes the problem.
Index: print-radius.c
===
RCS file: /cvs/src/usr.sbin/tcpdump/print-radius.c,v
retrieving
Hi,
the diff below is necessary to make suspend/resume work when x2apic is
enabled (i.e. on qemu/kvm/...). While I don't expect problems, it would be
nice if I could get some reports that this doesn't break suspend/resume on
real machines, especially older ones (like core 2 duo and older) and
fstat(2) mentioned twice under wpath.
utimes(2) referenced twice under fattr.
Rob
Index: tame.2
===
RCS file: /cvs/src/lib/libc/sys/tame.2,v
retrieving revision 1.21
diff -u -p -r1.21 tame.2
--- tame.2 26 Aug 2015 15:17:06
On Sun, Sep 06, 2015 at 11:44:11AM +0200, Stefan Fritsch wrote:
> Hi,
>
> the diff below is necessary to make suspend/resume work when x2apic is
> enabled (i.e. on qemu/kvm/...). While I don't expect problems, it would be
> nice if I could get some reports that this doesn't break suspend/resume
On Sun, Sep 06, 2015 at 02:30:43PM -0400, Rob Pierce wrote:
> fstat(2) mentioned twice under wpath.
>
> utimes(2) referenced twice under fattr.
>
> Rob
>
> Index: tame.2
> ===
> RCS file: /cvs/src/lib/libc/sys/tame.2,v
> retrieving
Don't allow negative numbers for font width, because it could lead to a
floating point exception (and wouldn't make sense anyway).
# wsfontload -w -7 /dev/null
Floating point exception (core dumped)
# _
While at it, use strtonum for proper error messages; also check font height.
Current error
11 matches
Mail list logo