Skip to content

Allow ECC Verify-only operations with NO_ASN and SP_MATH#2049

Merged
toddouska merged 1 commit intowolfSSL:masterfrom
danielinux:ecc-verify-only
Jan 21, 2019
Merged

Allow ECC Verify-only operations with NO_ASN and SP_MATH#2049
toddouska merged 1 commit intowolfSSL:masterfrom
danielinux:ecc-verify-only

Conversation

@danielinux
Copy link
Member

The lowest level of ECC signature verification does not require ASN. This patch allows to compile ECC with WOLFSSL_SP_MATH, removing the dependency on ASN module.

This is particularly useful in those embedded context where ECC signature verification is a requirement, but ASN is not an option due to resource restrictions (flash size).

Applications can now use wc_ecc_import_unsigned to import a public key and wc_ecc_verify_hash_ex to verify a signature, with no asn.c module compiled in.

Tested with wolfBoot ECDSA branch.

Originally based on the sp_cortexm branch (see dgarske#3 for previous related discussion)

cc: @dgarske @SparkiDev

@wolfSSL-Bot
Copy link

Can one of the admins verify this patch?

@dgarske
Copy link
Contributor

dgarske commented Jan 21, 2019

okay to test

Copy link
Contributor

@dgarske dgarske left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks Daniele. Once Jenkins is done I'd like to have Todd final review it.

@dgarske
Copy link
Contributor

dgarske commented Jan 21, 2019

OK to test

@dgarske dgarske assigned toddouska and unassigned danielinux Jan 21, 2019
Copy link
Contributor

@toddouska toddouska left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks

@toddouska toddouska merged commit d00dd9d into wolfSSL:master Jan 21, 2019
@danielinux danielinux deleted the ecc-verify-only branch January 22, 2019 01:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants