On Thu, May 20, 2021 at 12:15:54PM +0200, Marcus MERIGHI wrote:
> Hello Hiltjo,
>
> thanks for looking further than I did...
>
> Moving to tech@...
>
> hil...@codemadness.org (Hiltjo Posthuma), 2021.05.19 (Wed) 16:59 (CEST):
> > On Wed, May 19, 2021 at 01:20:34PM +0200, Marcus MERIGHI wrote:
> > > By accident I noticed that
> > >
> > > $ ssh -v $host
> > >
> > > gives me, among many other lines, this
> > >
> > > debug1: pledge: filesystem full
> > >
> > > Is this expected? Something to worry about?
> >
> > A grep shows its in the file /usr/src/usr.bin/ssh/clientloop.c client_loop()
> > function.
>
> to prevent the *very* similiar wording of the infamous dmesg output
>
> uid 539 on /var/db/clamav: file system full
>
> I'd suggest a slightly different message, something like this?
>
> Marcus
>
> Index: usr.bin/ssh/clientloop.c
> ===================================================================
> RCS file: /cvs/src/usr.bin/ssh/clientloop.c,v
> retrieving revision 1.363
> diff -u -p -u -r1.363 clientloop.c
> --- usr.bin/ssh/clientloop.c 19 May 2021 01:24:05 -0000 1.363
> +++ usr.bin/ssh/clientloop.c 20 May 2021 10:11:06 -0000
> @@ -1232,7 +1232,7 @@ client_loop(struct ssh *ssh, int have_pt
> fatal_f("pledge(): %s", strerror(errno));
>
> } else if (options.update_hostkeys) {
> - debug("pledge: filesystem full");
> + debug("pledge: filesystem full access");
> if (pledge("stdio rpath wpath cpath unix inet dns proc tty",
> NULL) == -1)
> fatal_f("pledge(): %s", strerror(errno));
Hi,
You're welcome. I think the patch makes sense.
--
Kind regards,
Hiltjo
