On Wed, May 18, 2016 at 12:02 PM, Stefan Beller <[email protected]> wrote: > void free_pathspec(struct pathspec *pathspec) > { > + int i, j; > + for (i = 0; i < pathspec->nr; i++) { > + for (j = 0; j < pathspec->items[j].attr_match_nr; j++) > + free(pathspec->items[i].attr_match[j].value); > + free(pathspec->items[i].attr_match); > + git_attr_check_free(pathspec->items[i].attr_check);
This is faulty as may be NULL and git_attr_check_free assumes its argument is not NULL. So I'll add a guard here in a reoll. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
