Skip to content

Commit e9ad531

Browse files
committed
stash: defensive programming
CodeQL points out that `parse_tree_indirect()` can return NULL values. Signed-off-by: Johannes Schindelin <[email protected]>
1 parent b5bb46b commit e9ad531

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

builtin/stash.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -343,7 +343,7 @@ static int reset_tree(struct object_id *i_tree, int update, int reset)
343343
memset(&opts, 0, sizeof(opts));
344344

345345
tree = parse_tree_indirect(i_tree);
346-
if (parse_tree(tree))
346+
if (!tree || parse_tree(tree))
347347
return -1;
348348

349349
init_tree_desc(t, &tree->object.oid, tree->buffer, tree->size);

0 commit comments

Comments
 (0)