From: Greg Beaver <cellog@php.net>
Date: Wed, 21 May 2008 20:24:40 +0000 (+0000)
Subject: fix convertTo\*() for same-named archives with no files
X-Git-Tag: BEFORE_NEW_PARAMETER_PARSE~172
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=79ae1c7273c321958febffb8bdf331ea899d4641;p=php

fix convertTo\*() for same-named archives with no files
---

diff --git a/ext/phar/phar_object.c b/ext/phar/phar_object.c
index 9c3f52434d..cc0156fb1c 100755
--- a/ext/phar/phar_object.c
+++ b/ext/phar/phar_object.c
@@ -1891,7 +1891,7 @@ static zval *phar_rename_archive(phar_archive_data *phar, char *ext, zend_bool c
 	efree(newname);
 
 	if (SUCCESS == zend_hash_find(&(PHAR_GLOBALS->phar_fname_map), newpath, phar->fname_len, (void **) &pphar)) {
-		if (*pphar == phar) {
+		if ((*pphar)->fname_len == phar->fname_len && !memcmp((*pphar)->fname, phar->fname, phar->fname_len)) {
 			if (!zend_hash_num_elements(&phar->manifest)) {
 				goto its_ok;
 			}
@@ -1929,7 +1929,7 @@ its_ok:
 		phar->alias_len = 0;
 	}
 
-	if (SUCCESS != zend_hash_add(&(PHAR_GLOBALS->phar_fname_map), newpath, phar->fname_len, (void*)&phar, sizeof(phar_archive_data*), NULL)) {
+	if (SUCCESS != zend_hash_update(&(PHAR_GLOBALS->phar_fname_map), newpath, phar->fname_len, (void*)&phar, sizeof(phar_archive_data*), NULL)) {
 		efree(oldpath);
 		zend_throw_exception_ex(spl_ce_BadMethodCallException, 0 TSRMLS_CC, "Unable to add newly converted phar \"%s\" to the list of phars", phar->fname);
 		return NULL;