From 42612e18d2e7c002054b677df791b848b62c1628 Mon Sep 17 00:00:00 2001 From: Phillip Wood Date: Mon, 26 Jun 2023 09:37:33 +0000 Subject: apply: improve error messages when reading patch Commit f1c0e3946e (apply: reject patches larger than ~1 GiB, 2022-10-25) added a limit on the size of patch that apply will process to avoid integer overflows. The implementation re-used the existing error message for when we are unable to read the patch. This is unfortunate because (a) it does not signal to the user that the patch is being rejected because it is too large and (b) it uses error_errno() without setting errno. This patch adds a specific error message for the case when a patch is too large. It also updates the existing message to make it clearer that it is the patch that cannot be read rather than any other file and marks both messages for translation. The "git apply" prefix is also dropped to match most of the rest of the error messages in apply.c (there are still a few error messages that prefixed with "git apply" and are not marked for translation after this patch). The test added in f1c0e3946e is updated accordingly. Reported-by: Premek Vysoky Signed-off-by: Phillip Wood Signed-off-by: Junio C Hamano --- apply.c | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) (limited to 'apply.c') diff --git a/apply.c b/apply.c index b17e8335b3..567bca96e7 100644 --- a/apply.c +++ b/apply.c @@ -398,9 +398,10 @@ static void say_patch_name(FILE *output, const char *fmt, struct patch *patch) static int read_patch_file(struct strbuf *sb, int fd) { - if (strbuf_read(sb, fd, 0) < 0 || sb->len >= MAX_APPLY_SIZE) - return error_errno("git apply: failed to read"); - + if (strbuf_read(sb, fd, 0) < 0) + return error_errno(_("failed to read patch")); + else if (sb->len >= MAX_APPLY_SIZE) + return error(_("patch too large")); /* * Make sure that we have some slop in the buffer * so that we can do speculative "memcmp" etc, and -- cgit v1.2.3