Merge branch 'js/ci-gcc-12-fixes'

This branch brings a couple of fixes to be able to compile Git in
`DEVELOPER=1` mode again.

The only remaining issue is a local variable's address in
`run_active_slot()` in `http.c` that is assigned to a struct that is
still valid after the function scope was left. Junio wanted to do this
differently from my proposed solution, and we'll merge Junio's
work-around next.

Signed-off-by: Johannes Schindelin <[email protected]>

Author: dscho

compat/nedmalloc/nedmalloc.c

@@ -323,7 +323,6 @@ static NOINLINE void RemoveCacheEntries(nedpool *p, threadcache *tc, unsigned in
 }
 static void DestroyCaches(nedpool *p) THROWSPEC
 {
-	if(p->caches)
 	{
 		threadcache *tc;
 		int n;

compat/win32/syslog.c

@@ -43,13 +43,15 @@ void syslog(int priority, const char *fmt, ...)
 	va_end(ap);
 
 	while ((pos = strstr(str, "%1")) != NULL) {
+		size_t offset = pos - str;
 		char *oldstr = str;
 		str = realloc(str, st_add(++str_len, 1));
 		if (!str) {
 			free(oldstr);
 			warning_errno("realloc failed");
 			return;
 		}
+		pos = str + offset;
 		memmove(pos + 2, pos + 1, strlen(pos));
 		pos[1] = ' ';
 	}

dir.c

@@ -3124,6 +3124,15 @@ char *git_url_basename(const char *repo, int is_bundle, int is_bare)
 			end--;
 	}
 
+	/*
+	 * It should not be possible to overflow `ptrdiff_t` by passing in an
+	 * insanely long URL, but GCC does not know that and will complain
+	 * without this check.
+	 */
+	if (end - start < 0)
+		die(_("No directory name could be guessed.\n"
+		      "Please specify a directory on the command line"));
+
 	/*
 	 * Strip trailing port number if we've got only a
 	 * hostname (that is, there is no dir separator but a