xgethostname: handle long hostnames
If the full hostname doesn't fit in the buffer supplied to gethostname, POSIX does not specify whether the buffer will be null-terminated, so to be safe, we should do it ourselves. Introduce new function, xgethostname, which ensures that there is always a \0 at the end of the buffer. Signed-off-by: David Turner <dturner@twosigma.com> Signed-off-by: Junio C Hamano <gitster@pobox.com>
This commit is contained in:

committed by
Junio C Hamano

parent
da25bdb776
commit
5781a9a270
@ -746,7 +746,7 @@ static int get_pack(struct fetch_pack_args *args,
|
||||
argv_array_push(&cmd.args, "--fix-thin");
|
||||
if (args->lock_pack || unpack_limit) {
|
||||
char hostname[HOST_NAME_MAX + 1];
|
||||
if (gethostname(hostname, sizeof(hostname)))
|
||||
if (xgethostname(hostname, sizeof(hostname)))
|
||||
xsnprintf(hostname, sizeof(hostname), "localhost");
|
||||
argv_array_pushf(&cmd.args,
|
||||
"--keep=fetch-pack %"PRIuMAX " on %s",
|
||||
|
Reference in New Issue
Block a user