[pacman-dev] [PATCH 3/6] lib/dload: code deduplication

Dave Reisner d at falconindy.com
Thu Aug 18 21:09:33 EDT 2011


Signed-off-by: Dave Reisner <dreisner at archlinux.org>
---
 lib/libalpm/dload.c |   10 +++++-----
 1 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/lib/libalpm/dload.c b/lib/libalpm/dload.c
index f8c10fa..a7a13b6 100644
--- a/lib/libalpm/dload.c
+++ b/lib/libalpm/dload.c
@@ -398,17 +398,17 @@ cleanup:
 	}
 
 	if(ret == 0) {
+		const char *realname = tempfile;
 		if (destfile) {
+			realname = destfile;
 			if(rename(tempfile, destfile)) {
 				_alpm_log(handle, ALPM_LOG_ERROR, _("could not rename %s to %s (%s)\n"),
 						tempfile, destfile, strerror(errno));
 				ret = -1;
-			} else if(final_file) {
-				STRDUP(*final_file, strrchr(destfile, '/') + 1,
-						RET_ERR(handle, ALPM_ERR_MEMORY, -1));
 			}
-		} else if(final_file) {
-			STRDUP(*final_file, strrchr(tempfile, '/') + 1,
+		}
+		if(ret != -1 && final_file) {
+			STRDUP(*final_file, strrchr(realname, '/') + 1,
 					RET_ERR(handle, ALPM_ERR_MEMORY, -1));
 		}
 	}
-- 
1.7.6



More information about the pacman-dev mailing list