diff options
author | Oliver Neukum <oneukum@suse.de> | 2007-07-23 08:58:39 +0200 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@suse.de> | 2007-07-30 13:27:45 -0700 |
commit | e31c18804f584dd838a752f6628e8c15bd7a3372 (patch) | |
tree | abead87733ffe5e3da890058d523851e849794f1 /drivers/usb/core/message.c | |
parent | 209b3cfd538e7d56d228cf6daf0b27e2cc26c6c2 (diff) | |
download | op-kernel-dev-e31c18804f584dd838a752f6628e8c15bd7a3372.zip op-kernel-dev-e31c18804f584dd838a752f6628e8c15bd7a3372.tar.gz |
USB: fix usb_serial_suspend(): buggy code
Am Montag 23 Juli 2007 schrieb Adrian Bunk:
> Commit ec22559e0b7a05283a3413bda5d177e42c950e23 added the following
> function to drivers/usb/serial/usb-serial.c:
>
[..]
>
> The Coverity checker spotted the inconsequent NULL checking for "serial".
>
> Looking at the code it also doesn't seem to have been intended to always
> return 0.
Coverity is right. The check for NULL is wrongly done and the error
return is lost.
Signed-off-by: Oliver Neukum <oneukum@suse.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'drivers/usb/core/message.c')
0 files changed, 0 insertions, 0 deletions