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 22:27:45 +0200 |
| commit | e31c18804f584dd838a752f6628e8c15bd7a3372 (patch) | |
| tree | abead87733ffe5e3da890058d523851e849794f1 /sound/synth | |
| parent | USB: yet another quirky device (diff) | |
| download | linux-e31c18804f584dd838a752f6628e8c15bd7a3372.tar.xz linux-e31c18804f584dd838a752f6628e8c15bd7a3372.zip | |
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 'sound/synth')
0 files changed, 0 insertions, 0 deletions