"brian m. carlson" <sand...@crustytoothpaste.net> 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 majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to