[Bug 432129] libusb-compat: f-spot crashes on first launch and when importing
https://bugzilla.novell.com/show_bug.cgi?id=432129
User sbrabec@novell.com added comment
https://bugzilla.novell.com/show_bug.cgi?id=432129#c16
Stanislav Brabec
Stanislav Brabec wrote:
Attached patch fixes crash, if USB is not initialized.
And attached update fixes even more crashes, if USB is not initialized. It replaces previous one.
Thanks for the patch!
Affected application: F-Spot Author: Marcus Meissner
Reference: https://bugzilla.novell.com/show_bug.cgi?id=432129
I have to sign up to see this bug report? From the 1 line description, it sounds like a F-Spot bug using libusb before it's initialized. I'm interested in matching the behaviour of libusb-0.1.12, but only if we do it precisely, which your patch doesn't quite do. For example, usb_close() on a NULL pointer will cause libusb-0.1.12 to crash, whereas your patch makes libusb-compat return -EINVAL. Also, nitpicking: there are some coding style flaws, please try and match the rest of the file. Some of the patch is fine, some uses spaces for indentation and has 'if' conditions and bodies on the same line. cheers Daniel -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
participants (1)
-
bugzilla_noreply@novell.com