git-send-email: fix fallback code in extract_valid_address()
authorKrzysztof Mazur <krzysiek@podlesie.net>
Thu, 22 Nov 2012 18:12:09 +0000 (19:12 +0100)
committerJunio C Hamano <gitster@pobox.com>
Mon, 26 Nov 2012 16:22:04 +0000 (08:22 -0800)
commit95c0d4b68a0fcba3adf45ce3a8f1edf99a9030a1
tree97d2cd6f758c5bb318926b86e8bca814cbc519d2
parent831a488b76e0fcad68c6d1c7aaf9f086199e1738
git-send-email: fix fallback code in extract_valid_address()

In the fallback check, used when Email::Valid is not available, the
extract_valid_address() uses $1 without checking for success of matching
regex. The $1 variable may still hold the result of previous match,
which is the address when email address was in '<>' or be undefined
otherwise.

Now if match fails undefined value is always returned to indicate error.
The same value is used by Email::Valid->address() in that case.

Previously 'foo@bar' address was rejected by Email::Valid and fallback,
but '<foo@bar>' was rejected by Email::Valid, but accepted by fallback.

Signed-off-by: Krzysztof Mazur <krzysiek@podlesie.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
git-send-email.perl