On 07/03/2015 08:47 AM, Philippe Leduc wrote:
> Hi,
>
> I noticed that digest return 0 even if the signature length is
> detected as incorrect. This patch also fix the documentation that
> seems obsolete (I considered that the patch was simply enough to make
> two corrections, but tell me if I should propose two patch).
>
>
> Subject: [PATCH] digest: digest doesn't return an error if the
> signature length is incorrect
>
> fix: digest return no error even if the digest length is wrong
> fix: documentation error (-v -> -s / -V -> -S)
>
> Signed-off-by: Philippe Leduc <ledphilippe@xxxxxxxxx>
> ---
> commands/digest.c | 5 +++--
> 1 file changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/commands/digest.c b/commands/digest.c
> index 340c07a..5304f32 100644
> --- a/commands/digest.c
> +++ b/commands/digest.c
> @@ -167,6 +167,7 @@ static int do_digest(int argc, char *argv[])
> } else if (siglen != digestlen) {
> eprintf("%s wrong size %zu, expected %zu\n",
> sigfile, siglen, digestlen);
> + ret = 1;
Better use COMMAND_ERROR here.
> goto err;
> }
> }
> @@ -186,8 +187,8 @@ BAREBOX_CMD_HELP_TEXT("Options:")
> BAREBOX_CMD_HELP_OPT ("-a <algo>\t", "hash or signature algorithm to use")
> BAREBOX_CMD_HELP_OPT ("-k <key>\t", "use supplied <key> (ASCII or
> hex) for MAC")
> BAREBOX_CMD_HELP_OPT ("-K <file>\t", "use key from <file> (binary) for MAC")
> -BAREBOX_CMD_HELP_OPT ("-v <hex>\t", "verify data against supplied
> <hex> (hash, MAC or signature)")
> -BAREBOX_CMD_HELP_OPT ("-V <file>\t", "verify data against <file>
> (hash, MAC or signature)")
> +BAREBOX_CMD_HELP_OPT ("-s <hex>\t", "verify data against supplied
> <hex> (hash, MAC or signature)")
> +BAREBOX_CMD_HELP_OPT ("-S <file>\t", "verify data against <file>
> (hash, MAC or signature)")
> BAREBOX_CMD_HELP_END
>
> BAREBOX_CMD_START(digest)
Your patch is line wrapped. Please consider using git send-email to send
your patches.
Marc
--
Pengutronix e.K. | Marc Kleine-Budde |
Industrial Linux Solutions | Phone: +49-231-2826-924 |
Vertretung West/Dortmund | Fax: +49-5121-206917-5555 |
Amtsgericht Hildesheim, HRA 2686 | http://www.pengutronix.de |
Attachment:
signature.asc
Description: OpenPGP digital signature
_______________________________________________ barebox mailing list barebox@xxxxxxxxxxxxxxxxxxx http://lists.infradead.org/mailman/listinfo/barebox
