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

[PATCH v3 7/9] http-push: check path length before using it

From
Tay Ray Chuan <rctay89@gmail.com>
Date
Nov 25, 2010, 08:21 UTC
Message-ID
<1290673270-4284-8-git-send-email-rctay89@gmail.com>
In-Reply-To
<1290673270-4284-7-git-send-email-rctay89@gmail.com>

We use path_len to skip the base url/path, but we do not know for sure if path does indeed contain the base url/path. Check if this is so.

Helped-by: Johnathan Nieder <jrnieder@gmail.com>
Signed-off-by: Tay Ray Chuan <rctay89@gmail.com>
---
 http-push.c |   12 ++++++++++--
 1 files changed, 10 insertions(+), 2 deletions(-)
diff --git a/http-push.c b/http-push.c
index 565e580..bfa1fe7 100644
--- a/http-push.c
+++ b/http-push.c
@@ -1116,8 +1116,16 @@ static void handle_remote_ls_ctx(struct xml_ctx *ctx, int tag_closed)
 				}
 			}
 			if (path) {
-				path += repo->path_len;
-				ls->dentry_name = xstrdup(path);
+				const char *url = repo->url;
+				if (repo->path)
+					url = repo->path;
+				if (strncmp(path, url, repo->path_len))
+					error("Parsed path '%s' does not match url: '%s'\n",
+					      path, url);
+				else {
+					path += repo->path_len;
+					ls->dentry_name = xstrdup(path);
+				}
 			}
 		} else if (!strcmp(ctx->name, DAV_PROPFIND_COLLECTION)) {
 			ls->dentry_flags |= IS_DIR;
-- 
1.7.3.2.495.gc7b3f
Previous: Tay Ray ChuanNext: Tay Ray Chuan
Message 10 of 12 in “Re: [PATCH v2 3/7] url: add str wrapper for end_url_with_slash()”
  1. Jonathan NiederNov 22, 2010
  2. Tay Ray ChuanNov 22, 2010
  3. 0/9 http: end urls with slashTay Ray Chuan, Nov 25, 2010
  4. 1/9 t5550-http-fetch: add missing '&&'Tay Ray Chuan, Nov 25, 2010
  5. 2/9 t5550-http-fetch: add test for http-fetchTay Ray Chuan, Nov 25, 2010
  6. 3/9 shift end_url_with_slash() from http.[ch] to url.[ch]Tay Ray Chuan, Nov 25, 2010
  7. 4/9 url: add str wrapper for end_url_with_slash()Tay Ray Chuan, Nov 25, 2010
  8. 5/9 http-backend: use end_url_with_slash()Tay Ray Chuan, Nov 25, 2010
  9. 6/9 http-push: Normalise directory names when pushing to some WebDAV serversTay Ray Chuan, Nov 25, 2010
  10. 7/9 http-push: check path length before using itTay Ray Chuan, Nov 25, 2010
  11. 8/9 http-push: add trailing slash at arg-parse time, instead of later onTay Ray Chuan, Nov 25, 2010
  12. 9/9 http-fetch: rework url handlingTay Ray Chuan, Nov 25, 2010

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.