From mboxrd@z Thu Jan 1 00:00:00 1970 From: Hannes Reinecke Date: Tue, 28 Sep 2004 07:03:21 +0000 Subject: [PATCH] NULL arg for get_device() / put_device() Message-Id: <41590CB9.8060405@suse.de> MIME-Version: 1 Content-Type: multipart/mixed; boundary="------------050905060005070201040605" List-Id: References: <415805DD.4040708@suse.de> <20040928012801.GA11125@kroah.com> In-Reply-To: <20040928012801.GA11125@kroah.com> To: Greg KH Cc: hotplug , Linux Kernel This is a multi-part message in MIME format. --------------050905060005070201040605 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: quoted-printable Greg KH wrote: > On Mon, Sep 27, 2004 at 02:21:49PM +0200, Hannes Reinecke wrote: >=20 >>Hi all, >> >>is there a specific reason that get_device accepts NULL as argument, >>whereas put_device() does not? >=20 >=20 > Um, I guess I never thought about it :) >=20 > I don't see why it wouldn't take it, feel free to send a patch. >=20 Here it is. Please apply. Cheers, Hannes --=20 Dr. Hannes Reinecke hare@suse.de SuSE Linux AG S390 & zSeries Maxfeldstra=DFe 5 +49 911 74053 688 90409 N=FCrnberg http://www.suse.de --------------050905060005070201040605 Content-Type: text/x-patch; name="null-argument-for-put_device.patch" Content-Transfer-Encoding: 7bit Content-Disposition: inline; filename="null-argument-for-put_device.patch" # This is a BitKeeper generated diff -Nru style patch. # # ChangeSet # 2004/09/28 08:59:54+02:00 hare@lammermuir.suse.de # Since get_device() accepts a NULL argument, put_device() should do so, too. # # Signed-off-by: Hannes Reinecke # # drivers/base/core.c # 2004/09/28 08:59:49+02:00 hare@lammermuir.suse.de +2 -1 # put_device() should accept a NULL argument. # diff -Nru a/drivers/base/core.c b/drivers/base/core.c --- a/drivers/base/core.c 2004-09-28 09:01:20 +02:00 +++ b/drivers/base/core.c 2004-09-28 09:01:20 +02:00 @@ -293,7 +293,8 @@ */ void put_device(struct device * dev) { - kobject_put(&dev->kobj); + if (dev) + kobject_put(&dev->kobj); } --------------050905060005070201040605-- ------------------------------------------------------- This SF.Net email is sponsored by: YOU BE THE JUDGE. Be one of 170 Project Admins to receive an Apple iPod Mini FREE for your judgement on who ports your project to Linux PPC the best. Sponsored by IBM. Deadline: Sept. 24. Go here: http://sf.net/ppc_contest.php _______________________________________________ Linux-hotplug-devel mailing list http://linux-hotplug.sourceforge.net Linux-hotplug-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/linux-hotplug-devel