Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 23 Mar 2015 02:08:36 +0100
From:      Mateusz Guzik <mjguzik@gmail.com>
To:        Rui Paulo <rpaulo@FreeBSD.org>
Cc:        svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org
Subject:   Re: svn commit: r278479 - in head: etc sys/kern
Message-ID:  <20150323010836.GC6798@dft-labs.eu>
In-Reply-To: <201502092313.t19NDpoS083043@svn.freebsd.org>
References:  <201502092313.t19NDpoS083043@svn.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Mon, Feb 09, 2015 at 11:13:51PM +0000, Rui Paulo wrote:
> Author: rpaulo
> Date: Mon Feb  9 23:13:50 2015
> New Revision: 278479
> URL: https://svnweb.freebsd.org/changeset/base/278479
> 
> Log:
>   Notify devd(8) when a process crashed.
>   
>   This change implements a notification (via devctl) to userland when
>   the kernel produces coredumps after a process has crashed.
>   devd can then run a specific command to produce a human readable crash
>   report.  The command is most usually a helper that runs gdb/lldb
>   commands on the file/coredump pair.  It's possible to use this
>   functionality for implementing automatic generation of crash reports.
>   
>   devd(8) will be notified of the full path of the binary that crashed and
>   the full path of the coredump file.
> 

The more I look at this the more I'm convinced this is quite insecure.

At a minimum this should also grow a flag to decide whether notification
about jailed process crashes are allowed. Off by default.

As it is you pass a path leading to a jail, but that's inherently
untrusted and will lead to trouble.

-- 
Mateusz Guzik <mjguzik gmail.com>



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20150323010836.GC6798>