diff options
| author | Thomas Rast <trast@inf.ethz.ch> | 2013-06-21 13:38:00 +0200 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2013-06-21 08:36:07 -0700 |
| commit | 212eb96a96e256933a76e48d85fa4f500bed0dee (patch) | |
| tree | 114f51785e5003bb54b424f0bf5eabd574b53cc5 /builtin/apply.c | |
| parent | 0ce2e396ee9fb0fa07e8381b338e49859dbf03db (diff) | |
| download | git-212eb96a96e256933a76e48d85fa4f500bed0dee.tar.gz | |
apply: carefully strdup a possibly-NULL name
2901bbe (apply: free patch->{def,old,new}_name fields, 2012-03-21)
cleaned up the memory management of filenames in the patches, but
forgot that find_name_traditional() can return NULL as a way of saying
"I couldn't find a name".
That NULL unfortunately gets passed into xstrdup() next, resulting in
a segfault. Use null_strdup() so as to safely propagate the null,
which will let us emit the correct error message.
Reported-by: DevHC on #git
Signed-off-by: Thomas Rast <trast@inf.ethz.ch>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'builtin/apply.c')
| -rw-r--r-- | builtin/apply.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/builtin/apply.c b/builtin/apply.c index b4428ea34f..f69a864abe 100644 --- a/builtin/apply.c +++ b/builtin/apply.c @@ -898,7 +898,7 @@ static void parse_traditional_patch(const char *first, const char *second, struc patch->old_name = name; } else { patch->old_name = name; - patch->new_name = xstrdup(name); + patch->new_name = null_strdup(name); } } if (!name) |