Martin Koegler <mkoegler@xxxxxxxxxxxxxxxxx> wrote: > +static int fsck_walk_commit(struct commit *commit, fsck_walk_func walk, void *data) > +{ > + struct commit_list *parents = commit->parents; > + int result; > + > + if(parse_commit(commit)) > + return -1; Hmm. Don't you need to get commit->parenst *after* it is parsed, and not before? > @@ -0,0 +1,10 @@ > +#ifndef GIT_FSCK_H > +#define GIT_FSCK_H > + > +#define OBJ_ANY OBJ_BAD Its unclear why this macro is necessary. -- Shawn. - To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html