Only free() after xstrdup()

1 files changed, 1 insertions, 2 deletions

diff --git a/noice.c b/noice.c
index e804e68..37cc91d 100644
--- a/noice.c
+++ b/noice.c
@@ -128,12 +128,11 @@ xdirname(const char *path)
 	 * original string if we lose track of it. */
 	tmp = xstrdup(path);
 	p = dirname(tmp);
-	free(tmp);
 	if (p == NULL)
 		printerr(1, "dirname");
-
 	/* Make sure this is a malloc(3)-ed string */
 	p = xstrdup(p);
+	free(tmp);
 	return p;
 }