git/list[1] front-page[2] threads[3] people[4] search[5] about
 

[PATCH] http-push.c: make CURLOPT_IOCTLDATA a usable pointer

From
AHAbbaad Haider <abbaad@gmail.com>
Date
Jul 6, 2014, 00:43 UTC
Message-ID
<53b89d7a.030d320a.0760.206f@mx.google.com>
In-Reply-To
<53b8918c.ca6d320a.7df8.18ac@mx.google.com>

Fixes a small bug affecting push to remotes which use some sort of multi-pass authentication. In particular the bug affected SabreDAV as configured by Box.com [1].

It must be a weird server configuration for the bug to have survived this long. Someone should write a test for it.

[1] http://marc.info/?l=git&m=140460482604482
Signed-off-by: Abbaad Haider <abbaad@gmail.com>
---
 http-push.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/http-push.c b/http-push.c
index f2c56c8..bd42895 100644
--- a/http-push.c
+++ b/http-push.c
@@ -199,7 +199,7 @@ static void curl_setup_http(CURL *curl, const char *url,
 	curl_easy_setopt(curl, CURLOPT_READFUNCTION, fread_buffer);
 #ifndef NO_CURL_IOCTL
 	curl_easy_setopt(curl, CURLOPT_IOCTLFUNCTION, ioctl_buffer);
-	curl_easy_setopt(curl, CURLOPT_IOCTLDATA, &buffer);
+	curl_easy_setopt(curl, CURLOPT_IOCTLDATA, buffer);
 #endif
 	curl_easy_setopt(curl, CURLOPT_WRITEFUNCTION, write_fn);
 	curl_easy_setopt(curl, CURLOPT_NOBODY, 0);
-- 
2.0.1
Previous: Abbaad HaiderNext: Jeff King
Message 2 of 3 in “[BUG] Push fails with dumb http to SabreDAV”
  1. Abbaad HaiderJul 6, 2014
  2. http-push.c: make CURLOPT_IOCTLDATA a usable pointerAbbaad Haider, Jul 6, 2014
  3. Jeff KingJul 12, 2014

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.