diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2022-05-25 15:30:13 +0300 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2022-06-10 11:48:19 +0200 |
commit | 62e4efe3375eb30292dabaec4481dc04550d3644 (patch) | |
tree | e31a2c9d1c0e37949478271c932f2bdec013b485 /net | |
parent | ecf6dedd03b03f05459c3662d9b141bedbc1c39d (diff) |
usbip: vudc: Don't enable IRQs prematurely
This code does:
spin_unlock_irq(&udc->ud.lock);
spin_unlock_irqrestore(&udc->lock, flags);
which does not make sense. In theory, the first unlock could enable
IRQs and then the second _irqrestore could disable them again. There
would be a brief momemt where IRQs were enabled improperly.
In real life, however, this function is always called with IRQs enabled
and the bug does not affect runtime.
Reviewed-by: Shuah Khan <skhan@linuxfoundation.org>
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Link: https://lore.kernel.org/r/Yo4hVWcZNYzKEkIQ@kili
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'net')
0 files changed, 0 insertions, 0 deletions