[PATCH] http-push: fix double free()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



There was an extra free(url) in remote_exists() (both if clauses before 
that contain a free(url) already), which resulted in a crash on Windows.

Noticed by Nigel Magnay.

Signed-off-by: Johannes Schindelin <johannes.schindelin@xxxxxx>
---

	This fixes Issue 71 of msysGit, as far as I can tell.

 http-push.c |    1 -
 1 files changed, 0 insertions(+), 1 deletions(-)

diff --git a/http-push.c b/http-push.c
index 64be904..a78e0a9 100644
--- a/http-push.c
+++ b/http-push.c
@@ -1991,7 +1991,6 @@ static int remote_exists(const char *path)
 		fprintf(stderr, "Unable to start HEAD request\n");
 	}
 
-	free(url);
 	return ret;
 }
 
-- 
1.5.3.6.2304.gbc8e45-dirty

-
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux