diff options
author | Jim Meyering <jim@meyering.net> | 2006-10-19 10:33:01 +0200 |
---|---|---|
committer | Junio C Hamano <junkio@cox.net> | 2006-10-19 01:49:26 -0700 |
commit | 7768e27e1d3f3d5e253e795433033b5de1d1c157 (patch) | |
tree | 852936c005eec280a6d2a9722eb1ace3f2315790 /builtin-apply.c | |
parent | Merge branch 'mw/pathinfo' (diff) | |
download | tgif-7768e27e1d3f3d5e253e795433033b5de1d1c157.tar.xz |
Don't use $author_name undefined when $from contains no /\s</.
I noticed a case not handled in a recent patch.
Demonstrate it like this:
$ touch new-file
$ git-send-email --dry-run --from j --to k new-file 2>err
new-file
OK. Log says:
Date: Thu, 19 Oct 2006 10:26:24 +0200
Sendmail: /usr/sbin/sendmail
From: j
Subject:
Cc:
To: k
Result: OK
$ cat err
Use of uninitialized value in pattern match (m//) at /p/bin/git-send-email line 416.
Use of uninitialized value in concatenation (.) or string at /p/bin/git-send-email line 420.
Use of uninitialized value in concatenation (.) or string at /p/bin/git-send-email line 468.
There's a patch for the $author_name part below.
The example above shows that $subject may also be used uninitialized.
That should be easy to fix, too.
Signed-off-by: Jim Meyering <jim@meyering.net>
Signed-off-by: Junio C Hamano <junkio@cox.net>
Diffstat (limited to 'builtin-apply.c')
0 files changed, 0 insertions, 0 deletions