Hi Shally,

> -----Original Message-----
> From: Shally Verma [mailto:[email protected]]
> Sent: Wednesday, May 16, 2018 7:05 AM
> To: De Lara Guarch, Pablo <[email protected]>
> Cc: Trahe, Fiona <[email protected]>; [email protected]; [email protected];
> [email protected]; Sunila Sahu <[email protected]>; 
> Ashish Gupta
> <[email protected]>
> Subject: [PATCH v3 5/6] crypto/openssl: add asym crypto support
> 
> Add asymmetric crypto operation support in openssl PMD.
> Current list of supported asym xforms:
> * RSA
> * DSA
> * Deffie-hellman
> * Modular Operations
> 
> changes from v2:
> - Update the pmd capability as per new capability structure
> 
> changes from v1:
> - resolve new line error in dod/guides/cryptodevs/openssl.rst
> 
> Signed-off-by: Shally Verma <[email protected]>
> Signed-off-by: Sunila Sahu <[email protected]>
> Signed-off-by: Ashish Gupta <[email protected]>


//snip//
> +/* process rsa operations */
> +static int process_openssl_rsa_op(struct rte_crypto_op *cop,
> +                               struct openssl_asym_session *sess)
> +{
> +     int ret = 0;
> +     struct rte_crypto_asym_op *op = cop->asym;
> +     RSA *rsa = sess->u.r.rsa;
> +     uint32_t pad = (op->rsa.pad);
> +
> +     switch (pad) {
> +     case RTE_CRYPTO_RSA_PKCS1_V1_5_BT1:
> +     case RTE_CRYPTO_RSA_PKCS1_V1_5_BT2:
> +             pad = RSA_PKCS1_PADDING;
> +             break;
> +     case RTE_CRYPTO_RSA_PADDING_PSS:
> +             pad = RSA_PKCS1_PSS_PADDING;
> +             /* fall through */
> +     case RTE_CRYPTO_RSA_PADDING_OAEP:
> +             pad = RSA_PKCS1_OAEP_PADDING;
> +             /* fall through */
> +     default:
> +             pad = RSA_NO_PADDING;
> +             break;
[Fiona] Maybe I'm missing something, but won't this fall through to 
RSA_NO_PADDING for OAEP and PSS cases?
 
//snip//

Reply via email to