linux-kernel - Re: get_device in device_create

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]

Date:	Wed, 2 Aug 2006 14:49:08 -0700
From:	Pete Zaitcev <zaitcev@...hat.com>
To:	Dmitry Torokhov <dtor@...ightbb.com>
Cc:	greg@...ah.com, linux-kernel@...r.kernel.org, zaitcev@...hat.com
Subject: Re: get_device in device_create_file

On Tue, 1 Aug 2006 21:13:05 -0400, Dmitry Torokhov <dtor@...ightbb.com> wrote:

> > > 	if (get_device(dev)) {
> > > 		error = sysfs_create_file(&dev->kobj, &attr->attr);
> > > 		put_device(dev);
> > > 	}

> > Buf it not, and the caller has a reference, then the call to
> > get_device is redundant.

> Yes it is. There are few of redundant gets and puts sprinkled around
> in the driver core, but the last time I mentioned that Greg was not
> quite ready to get rid of them ;)

I see a small, but nonzero harm from keeping them, for two reasons.
One, I reviewed a patch for RHEL-4 today, where the submitter copied
this code without thinking. So, if we don't flush these, they will
proliferate. Hackers assume that if Greg wrote that, it must be the
right way. Two, these calls shrink the race window and may be
masking something.

-- Pete
-
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/