Werner Koch <w...@gnupg.org> wrote:
>> +      /* Get VALUE.  */
>> +      value = sexp_nth_buffer (lvalue, 1, &valuelen);
>> +      if (!value)
>> +        rc = GPG_ERR_INV_OBJ;
>> +      if (rc)
>> +        goto leave;
>
> Please change the above to 
>
>> +      if (!value)
>> +        {
>> +          rc = GPG_ERR_INV_OBJ;
>> +          goto leave;
>> +        }
>
> just because this is easier to understand and the same pattern we use
> above.

I see.  Following your suggestion, applied those patches and pushed to master.
-- 

_______________________________________________
Gcrypt-devel mailing list
Gcrypt-devel@gnupg.org
https://lists.gnupg.org/mailman/listinfo/gcrypt-devel

Reply via email to