diff options
author | Stefan Naewe <stefan.naewe@gmail.com> | 2015-12-05 15:27:24 (GMT) |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2015-12-07 20:32:02 (GMT) |
commit | a2678df335f3404bc5d96683fa36b8cfab7946b0 (patch) | |
tree | bc5ec704370e60f8db15ad3f1dd409906bf5e7e9 /revision.c | |
parent | 24358560c3c0ab51c9ef8178d99f46711716f6c0 (diff) | |
download | git-a2678df335f3404bc5d96683fa36b8cfab7946b0.zip git-a2678df335f3404bc5d96683fa36b8cfab7946b0.tar.gz git-a2678df335f3404bc5d96683fa36b8cfab7946b0.tar.bz2 |
revision.c: fix possible null pointer arithmetic
mark_tree_uninteresting() dereferences a tree pointer before
checking if the pointer is valid. Fix that by doing the check first.
Signed-off-by: Stefan Naewe <stefan.naewe@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'revision.c')
-rw-r--r-- | revision.c | 4 |
1 files changed, 3 insertions, 1 deletions
@@ -131,10 +131,12 @@ static void mark_tree_contents_uninteresting(struct tree *tree) void mark_tree_uninteresting(struct tree *tree) { - struct object *obj = &tree->object; + struct object *obj; if (!tree) return; + + obj = &tree->object; if (obj->flags & UNINTERESTING) return; obj->flags |= UNINTERESTING; |