On 30.03.23 17:34, Alan Stern wrote:
Reference: https://syzkaller.appspot.com/bug?extid=23be03b56c5259385d79 The sisusbvga driver just assumes that the endpoints it uses will be present, without checking. I don't know anything about this driver, so the fix below may not be entirely correct.
Hi, this patch by itself looks good to me. But the need for it is problematic. Do we have any vendor specific driver that could get away without an equivalent to this patch without showing an equivalent bug? If so, why do we have a generic matching code, although it is always insufficient? What is the purpose of a generic binding interface in sysfs if every probe() method blocks it? Allowing a generic probe looks like a misdesign under these circumstances. You'd really want to add IDs to drivers. Regards Oliver
