uri: really count dots in verify_uri_segment()

This buggy implementation failed to allow "..." sequences, because the dot count was always zero. The usefulness of allowing "..." (or more dots) is debatable, but since it's a valid file name, we allow it.

uri: really count dots in verify_uri_segment()
cd21cfc1 · Max Kellermann · 05703cf7 · cd21cfc1
Commit cd21cfc1 authored Jun 25, 2010 by Max Kellermann
Hide whitespace changes
Inline Side-by-side

Showing with 4 additions and 1 deletion

uri.c src/uri.c +4 -1

No files found.
--- a/src/uri.c
+++ b/src/uri.c
@@ -52,8 +52,11 @@ verify_uri_segment(const char *p)
 	const char *q;

 	unsigned dots = 0;
-	while (*p == '.')
+	while (*p == '.') {
 		++p;
+		++dots;
+	}
+
 	if (dots <= 2 && (*p == 0 || *p == '/'))
 		return NULL;