[dpdk-dev] examples/ipsec-secgw: print correct crypto name

Message ID 1518172805-17461-1-git-send-email-radu.nicolau@intel.com (mailing list archive)
State Accepted, archived
Headers

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/Intel-compilation success Compilation OK

Commit Message

Radu Nicolau Feb. 9, 2018, 10:40 a.m. UTC
  When AES-256 was used aes-128 was printed in the console

Fixes: fa9088849e12 ("examples/ipsec-secgw: support AES 256")
Signed-off-by: Radu Nicolau <radu.nicolau@intel.com>
---
 examples/ipsec-secgw/sa.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)
  

Comments

Anoob Joseph Feb. 12, 2018, 7:02 a.m. UTC | #1
On 09/02/18 16:10, Radu Nicolau wrote:
> When AES-256 was used aes-128 was printed in the console
>
> Fixes: fa9088849e12 ("examples/ipsec-secgw: support AES 256")
> Signed-off-by: Radu Nicolau <radu.nicolau@intel.com>
Acked-by: Anoob Joseph <anoob.joseph@caviumnetworks.com>
> ---
>   examples/ipsec-secgw/sa.c | 3 ++-
>   1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/examples/ipsec-secgw/sa.c b/examples/ipsec-secgw/sa.c
> index 21239dd..d9dcc0e 100644
> --- a/examples/ipsec-secgw/sa.c
> +++ b/examples/ipsec-secgw/sa.c
> @@ -631,7 +631,8 @@ print_one_sa_rule(const struct ipsec_sa *sa, int inbound)
>   	printf("\tspi_%s(%3u):", inbound?"in":"out", sa->spi);
>   
>   	for (i = 0; i < RTE_DIM(cipher_algos); i++) {
> -		if (cipher_algos[i].algo == sa->cipher_algo) {
> +		if (cipher_algos[i].algo == sa->cipher_algo &&
> +				cipher_algos[i].key_len == sa->cipher_key_len) {
>   			printf("%s ", cipher_algos[i].keyword);
>   			break;
>   		}
  
Thomas Monjalon Feb. 13, 2018, 6:08 p.m. UTC | #2
12/02/2018 08:02, Anoob Joseph:
> 
> On 09/02/18 16:10, Radu Nicolau wrote:
> > When AES-256 was used aes-128 was printed in the console
> >
> > Fixes: fa9088849e12 ("examples/ipsec-secgw: support AES 256")
> > Signed-off-by: Radu Nicolau <radu.nicolau@intel.com>
> Acked-by: Anoob Joseph <anoob.joseph@caviumnetworks.com>

Applied, thanks
  

Patch

diff --git a/examples/ipsec-secgw/sa.c b/examples/ipsec-secgw/sa.c
index 21239dd..d9dcc0e 100644
--- a/examples/ipsec-secgw/sa.c
+++ b/examples/ipsec-secgw/sa.c
@@ -631,7 +631,8 @@  print_one_sa_rule(const struct ipsec_sa *sa, int inbound)
 	printf("\tspi_%s(%3u):", inbound?"in":"out", sa->spi);
 
 	for (i = 0; i < RTE_DIM(cipher_algos); i++) {
-		if (cipher_algos[i].algo == sa->cipher_algo) {
+		if (cipher_algos[i].algo == sa->cipher_algo &&
+				cipher_algos[i].key_len == sa->cipher_key_len) {
 			printf("%s ", cipher_algos[i].keyword);
 			break;
 		}