undef/git-commit-vandalism
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

fetch_bundle_uri(): drop pointless NULL check

Browse Source 
We check if "uri" is NULL, but it cannot be since we'd have segfaulted
earlier in the function when we unconditionally called xstrdup() on it.

In theory we might want to soften that xstrdup() to handle this case,
but even before the code which added it via c23f592117 (bundle-uri:
fetch a list of bundles, 2022-10-12), we'd have fed NULL to
fetch_bundle_uri_internal(), which would also segfault.

The extra check isn't hurting anything, but it does cause Coverity to
complain, and it may mislead somebody reading the code into thinking
that a NULL uri is something we're prepared to handle.

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
This commit is contained in:
Jeff King 2023-04-22 09:56:46 -04:00 committed by Junio C Hamano
parent ae6f064fd7
commit 0b1a95ef79
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
bundle-uri.c
View File

@ -795,10 +795,10 @@ int fetch_bundle_uri(struct repository *r, const char *uri,
init_bundle_list(&list); init_bundle_list(&list);
/* /*
* Do not fetch a NULL or empty bundle URI. An empty bundle URI * Do not fetch an empty bundle URI. An empty bundle URI
* could signal that a configured bundle URI has been disabled. * could signal that a configured bundle URI has been disabled.
*/ */
if (!uri || !*uri) { if (!*uri) {
result = 0; result = 0;
goto cleanup; goto cleanup;
} }