linux-kernel - Re: [PATCH] driver core: delete virtual directory on class

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <20061120203440.GA5458@suse.de>
Date:	Mon, 20 Nov 2006 12:34:40 -0800
From:	Greg KH <gregkh@...e.de>
To:	Akinobu Mita <akinobu.mita@...il.com>,
	Jiri Slaby <jirislaby@...il.com>,
	Linux kernel mailing list <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] driver core: delete virtual directory on class_unregister()

On Tue, Nov 21, 2006 at 04:53:18AM +0900, Akinobu Mita wrote:
> Class virtual directory is created as the need arises.
> But it is not deleted when the class is unregistered.
> 
> Cc: Greg Kroah-Hartman <gregkh@...e.de>
> Signed-off-by: Akinobu Mita <akinobu.mita@...il.com>
> 
> ---
>  drivers/base/class.c |    2 ++
>  1 file changed, 2 insertions(+)
> 
> Index: work-fault-inject/drivers/base/class.c
> ===================================================================
> --- work-fault-inject.orig/drivers/base/class.c
> +++ work-fault-inject/drivers/base/class.c
> @@ -163,6 +163,8 @@ int class_register(struct class * cls)
>  void class_unregister(struct class * cls)
>  {
>  	pr_debug("device class '%s': unregistering\n", cls->name);
> +	if (cls->virtual_dir)
> +		kobject_unregister(cls->virtual_dir);
>  	remove_class_attrs(cls);
>  	subsystem_unregister(&cls->subsys);
>  }

Hm, why is this not reproducable for me then without this patch?

Very strange, I'll dig into it some more this evening, gotta go do
family stuff for the rest of the day...

thanks,

greg k-h
-
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/