From d5addcf522deb05d259ecbc0946584d977879565 Mon Sep 17 00:00:00 2001 From: Jeff King Date: Wed, 21 Jun 2017 15:26:36 -0400 Subject: [PATCH] add--interactive: handle EOF in prompt_yesno The prompt_yesno function loops indefinitely waiting for a "y" or "n" response. But it doesn't handle EOF, meaning that we can end up in an infinite loop of reading EOF from stdin. One way to simulate that is with: echo e | GIT_EDITOR='echo corrupt >' git add -p Let's break out of the loop and propagate the undef to the caller. Without modifying the callers that effectively turns it into a "no" response. This is reasonable for both of the current callers, and it leaves room for any future caller to check for undef explicitly. Signed-off-by: Jeff King Signed-off-by: Junio C Hamano --- git-add--interactive.perl | 1 + 1 file changed, 1 insertion(+) diff --git a/git-add--interactive.perl b/git-add--interactive.perl index 4e0ab5a9bc..7c95324737 100755 --- a/git-add--interactive.perl +++ b/git-add--interactive.perl @@ -1152,6 +1152,7 @@ sub prompt_yesno { while (1) { print colored $prompt_color, $prompt; my $line = prompt_single_character; + return undef unless defined $line; return 0 if $line =~ /^n/i; return 1 if $line =~ /^y/i; } -- 2.11.4.GIT