When allocating a btrfs_device structure, device_list_add()
in volumes.c was not checking if the call to duplicate the
label string succeeded or not.

Signed-off-by: Filipe David Borba Manana <fdman...@gmail.com>
---
 volumes.c |    5 +++++
 1 file changed, 5 insertions(+)

diff --git a/volumes.c b/volumes.c
index d6f81f8..ea1d401 100644
--- a/volumes.c
+++ b/volumes.c
@@ -125,6 +125,11 @@ static int device_list_add(const char *path,
                        return -ENOMEM;
                }
                device->label = kstrdup(disk_super->label, GFP_NOFS);
+               if (!device->label) {
+                       kfree(device->name);
+                       kfree(device);
+                       return -ENOMEM;
+               }
                device->total_devs = btrfs_super_num_devices(disk_super);
                device->super_bytes_used = btrfs_super_bytes_used(disk_super);
                device->total_bytes =
-- 
1.7.9.5

--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to