On Sat, Jan 21, 2012 at 9:28 PM, Marcel Holtmann [off-list ref] wrote:
so instead of keep checking sock->sk multiple times in different places,
why not actually check the error from bt_proto[]->create() and skip the
reclassify_lock call.
And if we wanna be really paranoid, then lets add a BUG_ON for sk so we
get some nice backtraces if we have a wrongful caller.
Sounds like a good idea, I've sent v3 with the check removed.
Thanks,
Tavi