summary |
shortlog |
log |
commit | commitdiff |
tree
raw |
patch |
inline | side by side (from parent 1:
acf5957)
Clean up memory and file descriptor usage
Signed-off-by: Nick Hengeveld <nickh@reactrix.com>
Signed-off-by: Junio C Hamano <junkio@cox.net>
} else {
request->state = ABORTED;
free(request->url);
} else {
request->state = ABORTED;
free(request->url);
} else {
request->state = ABORTED;
free(request->url);
} else {
request->state = ABORTED;
free(request->url);
request->buffer.size = stream.total_out;
request->buffer.posn = 0;
request->buffer.size = stream.total_out;
request->buffer.posn = 0;
- if (request->url != NULL)
- free(request->url);
request->url = xmalloc(strlen(remote->url) +
strlen(request->lock->token) + 51);
strcpy(request->url, remote->url);
request->url = xmalloc(strlen(remote->url) +
strlen(request->lock->token) + 51);
strcpy(request->url, remote->url);
} else {
request->state = ABORTED;
free(request->url);
} else {
request->state = ABORTED;
free(request->url);
} else {
request->state = ABORTED;
free(request->url);
} else {
request->state = ABORTED;
free(request->url);
if (request->headers != NULL)
curl_slist_free_all(request->headers);
if (request->headers != NULL)
curl_slist_free_all(request->headers);
+
+ /* URL is reused for MOVE after PUT */
+ if (request->state != RUN_PUT) {
+ free(request->url);
+ request->url = NULL;
+ }
+
if (request->state == RUN_HEAD) {
if (request->http_code == 404) {
request->state = NEED_PUSH;
if (request->state == RUN_HEAD) {
if (request->http_code == 404) {
request->state = NEED_PUSH;
entry->next = entry->next->next;
}
entry->next = entry->next->next;
}
+ if (request->url != NULL)
+ free(request->url);
return error("Unable to start request");
}
return error("Unable to start request");
}