in the
getopt-loop, as I would most likely do more harm than good trying to
rewrite error().
The syslog(3) line, when suspending, was too long and corrected now.
Jesper Wallin
Index: usr.sbin/apmd/apmd.8
===
RCS file: /cvs/src/usr
On Tue, Aug 15, 2017 at 07:24:47PM -0400, Ted Unangst wrote:
> this looks good. will commit soon.
>
> (one nit, i'll correct, is man page options are upper case first.)
Noted for future reference. Thanks!
but two (non functionality) nits
> inline for future reference.
Thanks for the feedback, greatly appreciated! I've done a few changes
and also rewrote some of the manual, as my previous patch was lying.
Jesper Wallin
Index: usr.sbin/apmd/apmd.8
the specified percentage.
Jesper Wallin
Index: usr.sbin/apmd/apmd.8
===
RCS file: /cvs/src/usr.sbin/apmd/apmd.8,v
retrieving revision 1.47
diff -u -p -r1.47 apmd.8
--- usr.sbin/apmd/apmd.812 Feb 2015 14:03:49 - 1.47
Hi all,
I was reading through the code for less/filename.c and noticed that the
calling for seek_filesize() in filesize() is superfluous? A wild guess
is that it's remains from when BAD_LSEEK was removed?
Jesper Wallin
Index: usr.bin/less/filename.c
hing files.
Jesper Wallin
Index: fetch.c
===
RCS file: /cvs/src/usr.bin/ftp/fetch.c,v
retrieving revision 1.163
diff -u -p -r1.163 fetch.c
--- fetch.c 7 Mar 2017 08:00:23 - 1.163
+++ fetch.c 22 Sep 2017 19:5
Yeah, the SHA256.sig should be enough and just veify that everything is
correct. If not, fetch the incorrect sets again. I just wanted to give
a brief explaination on how I noticed this inconsistency. But yeah,
thanks for pointing me to a better solution.
On Sat, Sep 23, 2017 at 05:12:32AM
Yeah, fully understandable. I'll bump this once 6.2 is out.
I'll apply the changes Philip suggested (thanks!) and then build a
release to verify that everything works.
On Fri, Sep 22, 2017 at 11:35:03PM +0100, Stuart Henderson wrote:
> I like this, but it's rather late in the release cycle to
> On Sat, Sep 23, 2017 at 09:43:12AM +0200, Jesper Wallin wrote:
> Yeah, fully understandable. I'll bump this once 6.2 is out.
>
> I'll apply the changes Philip suggested (thanks!) and then build a
> release to verify that everything works.
>
>
Woups, seems like I m
Hi all,
The mentioning of the S option in malloc.conf(5) won't actually tell the
user what flags it enables. Rephrased this and list the options.
Jesper Wallin
Index: ./share/man/man5/malloc.conf.5
===
RCS file: /cvs/src/share
On Mon, Oct 16, 2017 at 07:51:47PM +0200, Anton Lindqvist wrote:
> On Mon, Oct 16, 2017 at 06:10:02PM +0100, Jason McIntyre wrote:
> > On Mon, Oct 16, 2017 at 06:38:31PM +0200, Anton Lindqvist wrote:
> > >
> > > Good call. Without knowing too much about the apmd internals, here's a
> > > first
), when things like lpd(8), smtpd(8) and packages
are.
The patch below adds a small section, pointing the admin to the pf(4)
and pf.conf(5) man pages.
Jesper Wallin
Index: afterboot.8
===
RCS file: /cvs/src/share/man/man8/afterboot
ent away. But yeah, it's also
stupid to bloat /bsd.rd with a fix for an uncommon cases like this. :-)
Jesper Wallin
On Fri, Sep 08, 2017 at 12:29:57PM +0100, Raf Czlonka wrote:
>
> One small comment regarding your patch - since a good while, sed(1)
> can do in place file edits so the above is a useless use of cat :^)
>
Hehe, that too. Thanks for pointing it out!
It's been a long week with way too little
name instead, before givig the
https error, but seeing that host, dig or nslookup is unavailable, I
decided to go with this instead.
Jesper Wallin
Index: distrib/miniroot/install.sub
===
RCS file: /cvs/src/distrib/miniroot/install.sub,v
or otherwise error out.
Jesper Wallin
Index: sbin/kbd/main.c
===
RCS file: /cvs/src/sbin/kbd/main.c,v
retrieving revision 1.8
diff -u -p -r1.8 main.c
--- sbin/kbd/main.c 30 Dec 2007 13:50:43 - 1.8
+++ sbin/kbd/main.c 29 Aug
On Wed, Aug 30, 2017 at 08:14:16AM +0200, Jeremie Courreges-Anglas wrote:
>
> Well, the behavior of kbd depends on the permissions of the device
> files, not on the fact that you're root or not. Maybe something like
> this would be more appropriate?
>
Ah, you're absolutely right! No idea why,
On Sat, Sep 23, 2017 at 11:50:46PM +0200, Jesper Wallin wrote:
> Woups, seems like I managed to break ftp(1) in the installer due to
> pledge being a bit too tight. Here is an updated version of the patch
> and with Philips changes as well.
...and hopefully a final version, sorry for
the console, but it's not possible to stop the machine since the vmd
owner is root.
The patch below simply leave vm->vm_uid untouched if the keeptty is set.
Jesper Wallin
Index: vmd.c
===
RCS file: /cvs/src/usr.sbin/vmd/vmd.
Shameless bump, now when we're out of beta. :-)
On Sat, Sep 16, 2017 at 10:53:47PM +0200, Jesper Wallin wrote:
> Hi all,
>
> I was reading through the code for less/filename.c and noticed that the
> calling for seek_filesize() in filesize() is superfluous? A wild guess
> is th
On Mon, Sep 25, 2017 at 10:36:18AM +0200, Landry Breuil wrote:
> On Mon, Sep 25, 2017 at 10:19:16AM +0200, Jesper Wallin wrote:
> > Shameless bump, now when we're out of beta. :-)
>
> being out of beta means we're in release mode..
Oh! Of course, silly me. Apologize for the noise.
On Tue, Aug 15, 2017 at 07:24:47PM -0400, Ted Unangst wrote:
> this looks good. will commit soon.
>
> (one nit, i'll correct, is man page options are upper case first.)
Sorry if this comes on as a bit pushy, but any news here? I don't know
how 6.2-beta affects this, but maybe it would be nice to
Hi,
Now when syspatch(8) is available and has been tested for a few months,
perhaps mention it in afterboot(8) under the Errata section?
Jesper Wallin
Index: share/man/man8/afterboot.8
===
RCS file: /cvs/src/share/man/man8
Hi again,
Bumping this, as I assume it wasn't prioritized during the beta.
On Sun, Sep 24, 2017 at 06:01:00PM +0200, Jesper Wallin wrote:
> Hi all,
>
> If a machine is configured in vm.conf to have the owner of a regular
> user, the ownership of the machine is lost upon reboot and
...and another bump. :-) *ping sthen@*
On Sun, Sep 24, 2017 at 07:25:33PM +0200, Jesper Wallin wrote:
> On Sat, Sep 23, 2017 at 11:50:46PM +0200, Jesper Wallin wrote:
> > Woups, seems like I managed to break ftp(1) in the installer due to
> > pledge being a bit too tight. Her
...and a third bump, this time we're actually out of beta for real. ;-)
On Mon, Sep 25, 2017 at 10:19:16AM +0200, Jesper Wallin wrote:
> Shameless bump, now when we're out of beta. :-)
>
> On Sat, Sep 16, 2017 at 10:53:47PM +0200, Jesper Wallin wrote:
> > Hi all,
> >
>
lso moved the pledge calls back
to the main to make it easier to see what promises are set and where.
I've still not been able to test this using MIDI devices, but everything
else seems to work as far as I can tell.
Regards,
Jesper Wallin
Index: aucat.c
=
On Mon, May 07, 2018 at 03:30:19PM +0200, Jesper Wallin wrote:
> I've still not been able to test this using MIDI devices, but everything
> else seems to work as far as I can tell.
As Theo (tb@) kindly pointed out to me off-list, aucat(1) supports the
use of multiple -i and/or -o flags, wh
On Fri, May 04, 2018 at 12:20:02PM +0200, Alexandre Ratchov wrote:
> AFAIU, only "sndio" is needed in offline(), because files are already
> opened/created.
Ah! Yeha, of course.. :-)
Regards,
Jesper Wallin
Index: aucat.c
===
is meant to be used so far down in
a program, but I drop all pledges except stdio and audio once
sio_open(2) is finished.
Regards,
Jesper Wallin
Index: aucat.c
===
RCS file: /cvs/src/usr.bin/aucat/aucat.c,v
retrieving revision 1
se file headers, then configure the device. This is
> not trivial.
>
Yeah, I noticed this as well when I tried to do the
refactoring. Unfortunately, I've been a bit busy lately, but I might
give this another try whenever I have the time.
Regards,
Jesper Wallin
Sperling wrote:
> On Sun, Apr 29, 2018 at 03:39:07AM +0200, Jesper Wallin wrote:
> > Hi all,
> >
> > I recently learned that my AP behaves badly and I have packet loss when
> > the background scan is running.
>
> Hi Jesper,
>
> Can you please check
obvious.
Regards,
Jesper Wallin
Index: aucat.c
===
RCS file: /cvs/src/usr.bin/aucat/aucat.c,v
retrieving revision 1.172
diff -u -p -r1.172 aucat.c
--- aucat.c 7 Nov 2017 11:39:24 - 1.172
+++ aucat.c 3 May 2018 19:13:14
Hi,
Adding a -P to your command might of course be easier, but since you
have -P in rm(1) and -v was added recently, won't this do:
mv newfile "$(rm -Pv oldfile)"
Regards
Jesper Wallin
the background scan has other use-cases that I'm not aware of, if
so, I apologize in advance. The patch below simply check if a bssid is
specified and if so, skip the background scan.
Regards,
Jesper Wallin
Index: ieee80211.c
===
RCS file
. Thanks for the pointers! Brainfart at my end due to
lack of coffee.
Regards,
Jesper Wallin
Index: ieee80211_input.c
===
RCS file: /cvs/src/sys/net80211/ieee80211_input.c,v
retrieving revision 1.199
diff -u -p -r1.199 ieee8021
However, when I set the bssid and it aborts the scan, the debug message
of "end background scan" appears before the actual list of APs.
Regards,
Jesper Wallin
Index: ieee80211_input.c
===
RCS file: /cvs/src/sys/net80211/ieee8
er testing and get some sort of tcpdump
or so from the Mac perhaps and see how OpenBSD behaves compared to
another client.
Regards,
Jesper Wallin
ithout any notices or errors, even if the command fails.
Jesper Wallin
Index: ifconfig.c
===
RCS file: /cvs/src/sbin/ifconfig/ifconfig.c,v
retrieving revision 1.348
diff -u -p -r1.348 ifconfig.c
--- ifconfig.c 29 Aug 2017 21:10:20 -
t it also feels like a bug if apmd becomes
unresponsive if one doesn't make apm execute in the background.
The patch below adds the exception.
Regards,
Jesper Wallin
Index: apmd.c
===
RCS file: /cvs/src/usr.sbin/apmd/apmd.c,v
retrievi
monitor up
Check with tcpdump if you receive any control frames (ack, rts, cts):
$ doas tcpdump -i urtwn0 -n -v -y IEEE802_11_RADIO
Also, please make sure that the NIC works BSS ("normal") mode as well.
Jesper Wallin
Ind
On Thu, Jan 24, 2019 at 03:56:17PM +0800, Kevin Lo wrote:
>
> First off, thanks for the diff. Right, urtwn(4) doesn't see any control
> frames in monitor mode, I think we also need to set R92C_RCR_ADF,
> R92C_RCR_ACF,
> and R92C_RCR_AMF bits in R92C_RCR to accept data/ctrl/mgmt frames
Hi all,
Replace the reference of 'php5' with 'php' in ports(7) since 'php5' is
deprecated and no longer exists in the ports tree.
Jesper Wallin
Index: share/man/man7/ports.7
===
RCS file: /cvs/src/share/man/man7/ports.7,v
_F_INVALID error. I guess the error
could be moved to the function above, but then we don't know the
device name.
Jesper Wallin
Index: sys/dev/softraid.c
===
RCS file: /cvs/src/sys/dev/softraid.c,v
retrieving revision 1.394
diff -u
ere fix a
> lie that secure has an off parameter and inform the user that it can't
> be turned off again.
Nice! Your patch is more elegant and good point about not failing hard.
Jesper Wallin
feels
ugly and "too deep".
Jesper Wallin
Index: usr.bin/vi/common/main.c
===
RCS file: /cvs/src/usr.bin/vi/common/main.c,v
retrieving revision 1.41
diff -u -p -r1.41 main.c
--- usr.bin/vi/common/main.c10 Nov
Hi all,
The kern.bufcachepercent entry in sysctl(2) still claims the default
value is 80, even though it was changed to 20 recently.
Index: ./lib/libc/sys/sysctl.2
===
RCS file: /cvs/src/lib/libc/sys/sysctl.2,v
retrieving revision
) at pool_do_put+0x40
pool_put(81de50d0,fd823c04d7e0) at pool_put+0x5a
futex_requeue(ed8068f5fd0,7fff,0,0,2) at futex_requeue+0x150
sys_futex(800043f0eaa8,800044682370,8000446823d0) at sys_futex+0xec
syscall(800044682440) at syscall+0x389
Xsyscall(0,53,7f7d6b01,53,ed80c78e4d0,ed8068f5fd0) at Xsyscall+0x128
end of kernel
end trace frame: 0x7f7d6a10, count: 251
--8<--
Jesper Wallin
Hi all,
When using vi(1) with secure mode (-S), both 'proc' and 'exec' are
stripped from the pledge promise. This breaks the :pre[serve] command
as it uses fork(2). This is broken on 6.4, 6.5 and -current.
Re-add the 'proc' promise, even when running in secure mode.
Jesper Wallin
Index
On Fri, Jul 19, 2019 at 05:14:03PM -0400, Bryan Steele wrote:
> I suspect that in secure/-S mode, the :pre[serve] should either be
> disabled, or modified to stop calling sendmail. The mail it is sending
> is purely advisory, and should be easy to disable. See common/recover.c.
Oh, you're right.
hat the code actually does, before trying to fix it.
2. Test, test with ktrace, test again and then run more tests. ;-)
Thanks a *lot* for your wise pointers and your thorough analysis!
Jesper Wallin
On Mon, Jul 22, 2019 at 06:24:28PM +0200, Ingo Schwarze wrote:
> But make sure that doesn't cause bugs to not get reported at all
> because the process causes too much work or takes too long. :)
>
Oh yeah, no worries! :-)
*bump*
On Fri, May 24, 2019 at 07:17:29PM +0200, Jesper Wallin wrote:
> Hi tech@
>
> A few days ago, I decided to set kern.securelevel=2 just for the sake
> of locking down my machine as much as possible. Today I learned that
> this affects bioctl(8) when trying to attach a crypt
ects "none",
"legacy" or "auto".
My guess is that tls_config_set_dheparams() has been updated and the
manual for relayd.conf(5) has not. Here's a diff that hopefully solves
that.
Jesper Wallin
Index: relayd.conf.5
==
On Tue, Jul 07, 2020 at 12:18:20PM +1000, David Gwynne wrote:
> you might want to put the memset after the check to see if the malloc
> failed...
>
Oh, wow... *goes to stand in the corner of shame*
Tested on a "Intel Dual Band Wireless-AC 9260" rev 0x29, msix
(hw rev 0x320, fw ver 34.3125811985.0)
I seem to be getting "iwm0: fatal firmware error" a few seconds after
the 4-way handshake. I can send a few packets, so it sure connects
and all, but then it fails shortly after.
iwm0: begin
) though.
As a side note, I noticed that the default value of _res.options was the
same value as RES_DEFAULT, so I changed it to RES_DEFAULT instead, for
the sake of consistency.
Thoughts?
Yours,
Jesper Wallin
[0] https://marc.info/?t=14998384322=1=2
Index: include/resolv.h
On Wed, Jul 29, 2020 at 02:46:06PM +0200, Jeremie Courreges-Anglas wrote:
> On Mon, Jul 27 2020, Jesper Wallin wrote:
>
> [...]
>
> > I still think the RES_USE_AD option might be a useful though, for when
> > you want to decide on an application-l
On Mon, Jul 27, 2020 at 03:30:52AM +0200, Jeremie Courreges-Anglas wrote:
> On Sun, Jul 26 2020, Jesper Wallin wrote:
> > On Sat, Jul 25, 2020 at 02:01:06PM +0200, Jeremie Courreges-Anglas wrote:
> >>
> >> For those two reasons I think the feature should be opt-in.
>
y that my fix is the
prettiest, but I get an extra byte and zero out the buffer. Maybe
someone has a more elegant fix for this.
Yours,
Jesper Wallin
Index: clparse.c
===
RCS file: /cvs/src/sbin/dhclient/clparse.c,v
retrieving revision 1.1
fected by it.
Yours,
Jesper Wallin
ned that
it would break the 'combined' log parsing for Webalizer and GoAccess.
I have no idea how my patch would affect those, but we already encode
backslash, newline and tabs the same way so I guess they handle it well.
A more interesting question is, how do they handle double-quotes in the
refe
Ping?
On Sat, Aug 21, 2021 at 08:21:55PM +0200, Jesper Wallin wrote:
> Hi tech@
>
> I'm trying to add log message when the pf(4) state table becomes
> exhausted/full. After reading the code, I learned that it's using
> pool_sethardlimit(9) to manage the resources.
&g
Hi tech@
I'm trying to add log message when the pf(4) state table becomes
exhausted/full. After reading the code, I learned that it's using
pool_sethardlimit(9) to manage the resources.
Conveniently, pool_sethardlimit(9) takes the argument warnmess, which is
printed to the console when the
On Mon, Aug 02, 2021 at 08:28:00PM +0200, Florian Obser wrote:
> On 2021-07-28 23:02 +02, Jesper Wallin wrote:
> > Hi tech@
> >
> > I've setup my machine to use trunk(4) with re(4) and iwm(4) as failover,
> > to make life easier when switching between wired and wireless
On Mon, Aug 02, 2021 at 11:36:47PM +0200, Jesper Wallin wrote:
> On Mon, Aug 02, 2021 at 08:28:00PM +0200, Florian Obser wrote:
> > On 2021-07-28 23:02 +02, Jesper Wallin wrote:
> > > Hi tech@
> > >
> > > I've setup my machine to use trunk(4) with re(4) and iw
Hi tech@
I've setup my machine to use trunk(4) with re(4) and iwm(4) as failover,
to make life easier when switching between wired and wireless
networking. The wired network at home is on a different subnet from
the wireless network, so when I unplug the cable, a DHCP request is
required to get
I concur, my 8-cores laptop (amd64) is happy with this patch.
Mostly using ungoogled-chromium, mutt, ncspot, xterm along with a few
vmd guests as dev machines. No isuses to report.
Jesper Wallin
On Tue, Nov 08, 2022 at 09:55:36AM +, Stefan Hagen wrote:
> Martin Pieuchot wrote (2022-11
68 matches
Mail list logo