Julian Phillips <julian@xxxxxxxxxxxxxxxxx> writes: > +static void add_ref(const char *name, const unsigned char *sha1, > + int flag, struct ref_array *refs, > + struct ref_entry **new_entry) > { > int len; > - struct ref_list *entry; > + struct ref_entry *entry; > > /* Allocate it and add it in.. */ > len = strlen(name) + 1; > - entry = xmalloc(sizeof(struct ref_list) + len); > + entry = xmalloc(sizeof(struct ref) + len); This should be sizeof(struct ref_entry), no? There is another such misallocation in search_ref_array() where it prepares a temporary. -- 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