From: Thiago Farina <tfransosi@gmail.com>
Date: Sat, 06 Nov 2010 17:46:51 GMT
Subject: Re: [PATCH 5/5] tag: recognize rfc1991 signatures
Message-ID: <AANLkTi=ZmBdJWKj1z0HE=tFoftzDSJaU5=Ji_1ywcA8+@mail.gmail.com>
In-Reply-To: <553a88c4ac00e681e605b81f3ce10342a881ac2f.1289041051.git.git@drmicha.warpmail.net>

On Sat, Nov 6, 2010 at 9:04 AM, Michael J Gruber
<git@drmicha.warpmail.net> wrote:
> diff --git a/tag.c b/tag.c
> index 5f9626c..18a5142 100644
> --- a/tag.c
> +++ b/tag.c
> @@ -138,7 +138,8 @@ size_t parse_signature(const char *buf, unsigned long size)
>  {
>        char *eol;
>        size_t len = 0;
> -       while (len < size && prefixcmp(buf + len, PGP_SIGNATURE)) {
> +       while (len < size && prefixcmp(buf + len, PGP_SIGNATURE)
> +                       && prefixcmp(buf + len, PGP_MESSAGE)) {

nit: I think this && should be in the end of the previous line. (Not
sure what is the preferred style though, comments?).