Re: signals logged / [RFC] log out-of-virtual-memory events

From: Folkert van Heusden
Date: Sun Jun 10 2007 - 15:53:53 EST


> > >> >+ {
> > >> if (sig_fatal(t, sig)) {
> > >> >+ printk(KERN_WARNING "Sig %d send to %d owned by %d.%d
> > >(%s)\n",
> > >> s/send/sent/;
> > >> >+ sig, t -> pid, t -> uid, t -> gid, t -> comm);
> > >> t->pid, t->uid, t->gid, t->comm);
> > >
> >
> > Gargh ... why does this want to be in the *kernel*'s logs? In any case, can
> > you please make this KERN_INFO (or lower) instead of KERN_WARNING.
> Description:
> This patch adds code to the signal-sender making it log a message when
> an unhandled fatal signal will be delivered.

New version. This one also informs the user about the sende pid/uid of
the signal (when applicable).

Signed-of by: Folkert van Heusden <folkert@xxxxxxxxxxxxxx

--- linux/kernel/signal.c.org 2007-05-20 22:47:13.000000000 +0200
+++ linux/kernel/signal.c 2007-06-10 00:21:31.000000000 +0200
@@ -739,6 +739,18 @@
struct sigqueue * q = NULL;
int ret = 0;

+ /* unhandled fatal signals are logged */
+ if (sig_fatal(t, sig)) {
+ if (is_si_special(info))
+ printk(KERN_INFO "Sig %d sent to %d owned by %d.%d (%s)\n",
+ sig, t->pid, t->uid, t->gid, t->comm);
+ else
+ printk(KERN_INFO "Sig %d sent to %d owned by %d.%d (%s), sent by pid %d, uid %d\n",
+ sig, t->pid, t->uid, t->gid, t->comm,
+ info -> _sifields._kill._pid,
+ info -> _sifields._kill._uid);
+ }
+
/*
* fast-pathed signals for kernel-internal things like SIGSTOP
* or SIGKILL.


Folkert van Heusden

--
Feeling generous? -> http://www.vanheusden.com/wishlist.php
----------------------------------------------------------------------
Phone: +31-6-41278122, PGP-key: 1F28D8AE, www.vanheusden.com
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/