linux-kernel - Re: [PATCH] dynamic debug: move ddebug_remove_module() down into free

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:	Thu, 22 Jul 2010 09:07:48 +0200
From:	Thomas Renninger <trenn@...e.de>
To:	Andrew Morton <akpm@...ux-foundation.org>
Cc:	Jason Baron <jbaron@...hat.com>, linux-kernel@...r.kernel.org,
	andi@...stfloor.org, hare@...e.de, yehuda@...newdream.net,
	stable@...nel.org
Subject: Re: [PATCH] dynamic debug: move ddebug_remove_module() down into free_module()

On Wednesday 21 July 2010 23:34:23 Andrew Morton wrote:
> On Wed, 14 Jul 2010 16:00:57 -0400
> Jason Baron <jbaron@...hat.com> wrote:
> 
> > move the call to ddebug_remove_module() down into free_module(). In this way
> > it should be called from all error paths. Currently, we are missing the
> > remove if the module init routine fails.
> > 
> > Signed-off-by: Jason Baron <jbaron@...hat.com>
> > Reported-by: Thomas Renninger <trenn@...e.de>
> > Tested-by: Thomas Renninger <trenn@...e.de>
> > CC: akpm@...ux-foundation.org
> > CC: andi@...stfloor.org
> > CC: hare@...e.de
> > CC: trenn@...e.de
> > CC: yehuda@...newdream.net
> > CC: linux-kernel@...r.kernel.org
> 
> Crappy changelog doesn't describe the bug we're fixing.  It should.
> 
> It's
> 
> 	echo "file ec.c +p" >/sys/kernel/debug/dynamic_debug/control

> causes an oops, yes?
AFAIK the query written doesn't matter.
It's about the declaration of specific modules (see below).
If you compile out (a) specific module(s), above query might even
work.
 
... 
> Should we backport this into 2.6.34.x?  Earlier?
We saw oopses in 2.6.32 due to a module without an exit function
and overwriting the pr_fmt macro (not 100% sure whether both
conditions must be met, according to Hannes' test yes).
I expect the patch should go at least back to this one.
I can give it a try then whether this also fixes this condition.

      Thomas
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/