Re: [PATCH 1/1] Fix recipient santitization
From: Junio C Hamano <hidden>
Date: 2016-06-15 22:44:38
Last night I was going through old mail-logs and found this, which I think still is needed. Does anybody see anything wrong with the patch (aside from missing full-stop at the end of the sentence)? "Horst H. von Brand" [off-list ref] writes:
quoted hunk
Need to quote all special characters, not just the first one Signed-off-by: Horst H. von Brand <redacted> --- git-send-email.perl | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-)diff --git a/git-send-email.perl b/git-send-email.perl index f62f119..3bdd225 100755 --- a/git-send-email.perl +++ b/git-send-email.perl@@ -635,7 +635,7 @@ sub sanitize_address # double quotes are needed if specials or CTLs are included elsif ($recipient_name =~ /[][()<>@,;:\\".\000-\037\177]/) { - $recipient_name =~ s/(["\\\r])/\\$1/; + $recipient_name =~ s/(["\\\r])/\\$1/g; $recipient_name = "\"$recipient_name\""; }-- 1.5.5.rc2