From: Jesper Juhl <[EMAIL PROTECTED]>

We may return from drivers/usb/serial/whiteheat.c::whiteheat_attach()
without freeing `result' if we leave via the no_firmware: label.

Spotted by the coverity checker as #670

Signed-off-by: Jesper Juhl <[EMAIL PROTECTED]>
Signed-off-by: Greg Kroah-Hartman <[EMAIL PROTECTED]>

---

 drivers/usb/serial/whiteheat.c |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

67ca0284f69992ad71ac12dc375f2b158d9d703d
diff --git a/drivers/usb/serial/whiteheat.c b/drivers/usb/serial/whiteheat.c
index 557411c..f806553 100644
--- a/drivers/usb/serial/whiteheat.c
+++ b/drivers/usb/serial/whiteheat.c
@@ -508,6 +508,7 @@ no_firmware:
        err("%s: Unable to retrieve firmware version, try replugging\n", 
serial->type->description);
        err("%s: If the firmware is not running (status led not blinking)\n", 
serial->type->description);
        err("%s: please contact [EMAIL PROTECTED]", serial->type->description);
+       kfree(result);
        return -ENODEV;
 
 no_command_private:
-- 
1.3.1



-------------------------------------------------------
Using Tomcat but need to do more? Need to support web services, security?
Get stuff done quickly with pre-integrated technology to make your job easier
Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
_______________________________________________
linux-usb-devel@lists.sourceforge.net
To unsubscribe, use the last form field at:
https://lists.sourceforge.net/lists/listinfo/linux-usb-devel

Reply via email to