diff options
author | Erik Faye-Lund <kusmabite@gmail.com> | 2009-03-23 12:53:08 +0000 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2009-03-29 19:48:43 -0700 |
commit | c0582c53bcf4e83bba70e1ad23abbad31f96ebc8 (patch) | |
tree | bd5a8222f3352aba250a1777a764ccffc194e330 /builtin-fast-export.c | |
parent | 2d07f6d4b728d19d115b89123224268718f34efd (diff) | |
download | git-c0582c53bcf4e83bba70e1ad23abbad31f96ebc8.tar.gz |
builtin-fast-export.c: fix crash on tagged trees
If a tag object points to a tree (or another unhandled type), the commit-
pointer is left uninitialized and later dereferenced. This patch adds a
default case to the switch that issues a warning and skips the object.
Signed-off-by: Erik Faye-Lund <kusmabite@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'builtin-fast-export.c')
-rw-r--r-- | builtin-fast-export.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/builtin-fast-export.c b/builtin-fast-export.c index 8a6cf5e649..1ec459f057 100644 --- a/builtin-fast-export.c +++ b/builtin-fast-export.c @@ -375,6 +375,10 @@ static void get_tags_and_duplicates(struct object_array *pending, case OBJ_BLOB: handle_object(tag->object.sha1); continue; + default: + warning("Tag points to object of unexpected type %s, skipping.", + typename(tag->object.type)); + continue; } break; default: |