W dniu 16.01.2015 o 13:12, Dan Carpenter pisze:
Oh, yeah. You're right. I should have been more careful and I should
have seen that. Sorry. But the problem is the original code is still a
bit buggy.
We call:
data = kzalloc();
Inside __uvcg_fill_strm() we do "data += something;"
kfree(data);
We should save the orignal data pointer so that we can free it
correctly at the end in uvcg_streaming_class_allow_link().
Yeah, right. Thank you for spotting this. The kfree() is called
only if __uvcg_fill_strm() fails, though. But of course this needs
to be fixed.
AP
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html