Use strdup() instead of malloc(strlen())+strcpy()

Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

2 files changed, 4 insertions, 15 deletions

diff --git a/src/display.c b/src/display.c
index 7998fe7..77dd325 100644
--- a/src/display.c
+++ b/src/display.c
@@ -216,17 +216,8 @@ _XcursorGetDisplayInfo (Display *dpy)
 	v = XGetDefault (dpy, "Xcursor", "theme");
     if (v)
     {
-	int len;
-
-	len = strlen (v) + 1;
-
-	info->theme = malloc (len);
-	if (info->theme)
-	    strcpy (info->theme, v);
-
-	info->theme_from_config = malloc (len);
-	if (info->theme_from_config)
-	    strcpy (info->theme_from_config, v);
+	info->theme = strdup (v);
+	info->theme_from_config = strdup (v);
     }
 
     /*
@@ -342,10 +333,9 @@ XcursorSetTheme (Display *dpy, const char *theme)
 
     if (theme)
     {
-	copy = malloc (strlen (theme) + 1);
+	copy = strdup (theme);
 	if (!copy)
 	    return XcursorFalse;
-	strcpy (copy, theme);
     }
     else
 	copy = NULL;
diff --git a/src/file.c b/src/file.c
index ce9de78..43163c2 100644
--- a/src/file.c
+++ b/src/file.c
@@ -86,12 +86,11 @@ XcursorImagesSetName (XcursorImages *images, const char *name)
     if (!images || !name)
         return;
 
-    new = malloc (strlen (name) + 1);
+    new = strdup (name);
 
     if (!new)
 	return;
 
-    strcpy (new, name);
     if (images->name)
 	free (images->name);
     images->name = new;