> +     down_read(&usb_bus_type.subsys.rwsem);
> +     if (interface && interface->dev.driver) {
> +             strncpy(gd.driver, interface->dev.driver->name, sizeof(gd.driver));
> +             ret = copy_to_user(arg, &gd, sizeof(gd)) ? -EFAULT : 0;
> +     }
> +     up_read(&usb_bus_type.subsys.rwsem);
> +     return ret;

IMHO you should drop the lock before you copy to userspace.

        Regards
                Oliver



-------------------------------------------------------
This SF.Net email is sponsored by: IBM Linux Tutorials
Free Linux tutorial presented by Daniel Robbins, President and CEO of
GenToo technologies. Learn everything from fundamentals to system
administration.http://ads.osdn.com/?ad_id=1470&alloc_id=3638&op=click
_______________________________________________
[EMAIL PROTECTED]
To unsubscribe, use the last form field at:
https://lists.sourceforge.net/lists/listinfo/linux-usb-devel

Reply via email to