> Good catch, but unless I'm missing something from just looking at the
> patch rather than the whole 2.5 version of the code, it appears that the
> "usblp->minor >= USBLP_MINORS" error may be improperly handled here,
> because "usb_deregister_dev (&usblp_driver, 1, usblp->minor)" will
> be called
Oliver Neukum wrote:
> you fail to free minors if something goes wrong in probe.
...
> usblp->minor++;
> if (usblp->minor >= USBLP_MINORS) {
> err("no more free usblp devices");
> - goto abort;
>
Hi,
you fail to free minors if something goes wrong in probe.
Regards
Oliver
You can import this changeset into BK by piping this whole message to:
'| bk receive [path to repository]' or apply the patch as usual.
==
