]> git.ipfire.org Git - thirdparty/git.git/commit
clone.c: avoid "exceeds maximum object size" error with GCC v12.x
authorJohannes Schindelin <johannes.schindelin@gmx.de>
Tue, 24 May 2022 00:23:06 +0000 (00:23 +0000)
committerJohannes Schindelin <johannes.schindelin@gmx.de>
Wed, 22 Mar 2023 16:53:32 +0000 (17:53 +0100)
commit321854ac464bbc85ff2afe089c4a56505a1c9e25
treec4ea38b96ab7e23378554707f700b5239088603a
parent86f6f4fa91a9a2310c8f73528a86bf2c52d47fe4
clone.c: avoid "exceeds maximum object size" error with GCC v12.x

Technically, the pointer difference `end - start` _could_ be negative,
and when cast to an (unsigned) `size_t` that would cause problems. In
this instance, the symptom is:

dir.c: In function 'git_url_basename':
dir.c:3087:13: error: 'memchr' specified bound [9223372036854775808, 0]
       exceeds maximum object size 9223372036854775807
       [-Werror=stringop-overread]
    CC ewah/bitmap.o
 3087 |         if (memchr(start, '/', end - start) == NULL
      |             ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

While it is a bit far-fetched to think that `end` (which is defined as
`repo + strlen(repo)`) and `start` (which starts at `repo` and never
steps beyond the NUL terminator) could result in such a negative
difference, GCC has no way of knowing that.

See also https://gcc.gnu.org/bugzilla//show_bug.cgi?id=85783.

Let's just add a safety check, primarily for GCC's benefit.

Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
builtin/clone.c