summary refs log tree commit diff stats
path: root/hw/usb
diff options
context:
space:
mode:
authorGerd Hoffmann <kraxel@redhat.com>2014-04-25 12:01:55 +0200
committerGerd Hoffmann <kraxel@redhat.com>2014-05-05 12:57:45 +0200
commit2dc7fdf33d28940255f171b8ea4b692d9d5b7a7d (patch)
tree31d387d4c47ce98a4634feafe548a46c76fa659d /hw/usb
parent9e4eff5b54a5f99537420e75fe2c686ec430d32a (diff)
downloadfocaccia-qemu-2dc7fdf33d28940255f171b8ea4b692d9d5b7a7d.tar.gz
focaccia-qemu-2dc7fdf33d28940255f171b8ea4b692d9d5b7a7d.zip
usb: mtp: fix error path memory leak
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
Reviewed-by: Peter Wu <peter@lekensteyn.nl>
Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
Diffstat (limited to 'hw/usb')
-rw-r--r--hw/usb/dev-mtp.c2
1 files changed, 2 insertions, 0 deletions
diff --git a/hw/usb/dev-mtp.c b/hw/usb/dev-mtp.c
index 775dc8d0b2..45f9562c60 100644
--- a/hw/usb/dev-mtp.c
+++ b/hw/usb/dev-mtp.c
@@ -669,6 +669,7 @@ static MTPData *usb_mtp_get_object(MTPState *s, MTPControl *c,
 
     d->fd = open(o->path, O_RDONLY);
     if (d->fd == -1) {
+        usb_mtp_data_free(d);
         return NULL;
     }
     d->length = o->stat.st_size;
@@ -688,6 +689,7 @@ static MTPData *usb_mtp_get_partial_object(MTPState *s, MTPControl *c,
 
     d->fd = open(o->path, O_RDONLY);
     if (d->fd == -1) {
+        usb_mtp_data_free(d);
         return NULL;
     }