linux-kernel - Re: Re: pktcdvd oops

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 for Android: free password hash cracker in your pocket

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

Date:	Thu, 8 Nov 2007 18:49:48 +0100 (CET)
From:	Peter Osterlund <petero2@...ia.com>
To:	Thomas Maier <balagi@...tmail.de>
cc:	Tejun Heo <htejun@...il.com>, Jens Axboe <jens.axboe@...cle.com>,
	linux-kernel <linux-kernel@...r.kernel.org>, gregkh@...e.de
Subject: Re: Re: pktcdvd oops

On Thu, 8 Nov 2007, balagi@...tmail.de wrote:

> tested it too, running linux 2.6.23 in a qemu instance, and the patch 
> worked. But i would prefer to take the try_module_get() stuff into 
> pkt_setup_dev() because it is used also in the older procfs interface. 
> Can we run into the same problem here, means procfs holds no module 
> references too, like sysfs now?

The procfs interface can only be used to get some debug data out from the 
driver, not to bind the driver to a CD/DVD device, so it shouldn't be a 
problem.

The other way to bind a device is to use the pktsetup program, which is 
doing ioctl calls to the driver. In that case, user space has to open the 
device before being able to do the ioctls, and the open call will increase 
the reference count.

-- 
Peter Osterlund - petero2@...ia.com
http://web.telia.com/~u89404340
-
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/