commit be168af860109a1ecc8526c5bbe0ace1536448b8
author Markus Armbruster <armbru@redhat.com> Tue Jan 22 11:08:04 2013 +0100
committer Stefan Hajnoczi <stefanha@redhat.com> Wed Jan 30 11:14:46 2013 +0100
tree acb0700408c43b8205cff101ded58b1aa5ba62b3
parent 40a50b0a73d185c85cf62023f07e3091861081bb

libcacard: Fix unchecked strdup() by converting to g_strdup()

Note that we already free with g_free().

Signed-off-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>

libcacard/vreader.c

diff --git a/libcacard/vreader.c b/libcacard/vreader.c
index 313349b..f3efc27 100644
--- a/libcacard/vreader.c
+++ b/libcacard/vreader.c
@@ -49,7 +49,7 @@
     reader = (VReader *)g_malloc(sizeof(VReader));
     qemu_mutex_init(&reader->lock);
     reader->reference_count = 1;
-    reader->name = name ? strdup(name) : NULL;
+    reader->name = g_strdup(name);
     reader->card = NULL;
     reader->id = (vreader_id_t)-1;
     reader->reader_private = private;