bin: copy: Fix various resource leaks introduced by refactor

Thanks coverity! <3

Fixes: d26e311 ("bin: copy: Refactor")

Author: wkz

src/bin/copy.c

@@ -338,7 +338,7 @@ static int curl(char *op, const char *path, const char *uri)
 	char *argv[] =  {
 		"curl", "-L", op, NULL, NULL, NULL, NULL, NULL,
 	};
-	int err;
+	int err = 1;
 
 	argv[3] = strdup(path);
 	argv[4] = strdup(uri);
@@ -390,7 +390,7 @@ static int cp(const char *srcpath, const char *dstpath)
 	char *argv[] =  {
 		"cp", NULL, NULL, NULL,
 	};
-	int err;
+	int err = 1;
 
 	argv[1] = strdup(srcpath);
 	argv[2] = strdup(dstpath);
@@ -442,13 +442,20 @@ static int get(const char *src, const struct infix_ds *ds, const char *path)
 
 static int resolve_src(const char **src, const struct infix_ds **ds, char **path, bool *rm)
 {
+	int tmpfd;
+
 	*src = infix_ds(*src, ds);
 
 	if (*ds || is_uri(*src)) {
-		*path = tempnam(NULL, NULL);
+		*path = strdup("/tmp/copy-XXXXXX");
 		if (!*path)
 			return 1;
 
+		tmpfd = mkstemp(*path);
+		if (tmpfd < 0)
+			return 1;
+
+		close(tmpfd);
 		*rm = true;
 		return 0;
 	} else {
@@ -477,7 +484,7 @@ static int resolve_dst(const char **dst, const struct infix_ds **ds, char **path
 		if (!(*ds)->path)
 			return 0;
 
-		*path = (*ds)->path;
+		*path = strdup((*ds)->path);
 	} else if (is_uri(*dst)) {
 		return 0;
 	} else {
@@ -499,8 +506,8 @@ static int resolve_dst(const char **dst, const struct infix_ds **ds, char **path
 
 static int copy(const char *src, const char *dst)
 {
+	char *srcpath = NULL, *dstpath = NULL;
 	const struct infix_ds *srcds, *dstds;
-	char *srcpath, *dstpath;
 	bool rmsrc = false;
 	mode_t oldmask;
 	int err = 1;
@@ -534,6 +541,9 @@ static int copy(const char *src, const char *dst)
 	if (rmsrc)
 		rmtmp(srcpath);
 
+	free(dstpath);
+	free(srcpath);
+
 	sync();
 	umask(oldmask);
 	return err;

src/bin/erase.c

@@ -16,7 +16,7 @@ static int sanitize;
 static int do_erase(const char *name)
 {
 	char *path;
-	int rc;
+	int rc = 0;
 
 	path = cfg_adjust(name, NULL, sanitize);
 	if (!path) {
@@ -25,10 +25,8 @@ static int do_erase(const char *name)
 		goto out;
 	}
 
-	if (!yorn("Remove %s, are you sure?", path)) {
-		rc = 0;
+	if (!yorn("Remove %s, are you sure?", path))
 		goto out;
-	}
 
 	if (remove(path)) {
 		fprintf(stderr, ERRMSG "failed removing %s: %s\n", path, strerror(errno));