"brian m. carlson" <sandals@xxxxxxxxxxxxxxxxxxxx> writes:
> diff --git a/builtin/log.c b/builtin/log.c
> index dd8f3fc..e0465ba 100644
> --- a/builtin/log.c
> +++ b/builtin/log.c
> @@ -807,6 +807,12 @@ static void get_patch_ids(struct rev_info *rev, struct patch_ids *ids)
> o2 = rev->pending.objects[1].item;
> flags2 = o2->flags;
>
> + o1 = deref_tag(o1, NULL, 0);
> + o2 = deref_tag(o2, NULL, 0);
> +
> + if (!o1 || !o2)
> + die(_("Invalid tag."));
Shouldn't you ensure o1 and o2 are commits here?
--
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