patches for DPDK stable branches
 help / color / mirror / Atom feed
* [dpdk-stable] patch 'ip_frag: fix overflow in key comparison' has been queued to stable release 18.08.1
@ 2018-11-23 10:26 Kevin Traynor
  2018-11-23 10:26 ` [dpdk-stable] patch 'eal/linux: fix memory leak of logid' " Kevin Traynor
                   ` (67 more replies)
  0 siblings, 68 replies; 72+ messages in thread
From: Kevin Traynor @ 2018-11-23 10:26 UTC (permalink / raw)
  To: Li Han; +Cc: Konstantin Ananyev, dpdk stable

Hi,

FYI, your patch has been queued to stable release 18.08.1

Note it hasn't been pushed to http://dpdk.org/browse/dpdk-stable yet.
It will be pushed if I get no objections before 11/29/18. So please
shout if anyone has objections.

Also note that after the patch there's a diff of the upstream commit vs the patch applied
to the branch. If the code is different (ie: not only metadata diffs), due for example to
a change in context or macro names, please double check it.

Thanks.

Kevin Traynor

---
>From 362b57a19d3d606c23dd4ce198d608e34351b22f Mon Sep 17 00:00:00 2001
From: Li Han <han.li1@zte.com.cn>
Date: Wed, 1 Aug 2018 22:01:03 -0400
Subject: [PATCH] ip_frag: fix overflow in key comparison

[ upstream commit 8721e07478c6edc9565645047742f79c654cfd1b ]

in struct ip_frag_key,src_dst[] type is uint64_t.
but "val" which to store the calc restult ,type is uint32_t.
we may lost high 32 bit key. and function return value is int,
but it won't return < 0.

Signed-off-by: Li Han <han.li1@zte.com.cn>
Acked-by: Konstantin Ananyev <konstantin.ananyev@intel.com>
---
 lib/librte_ip_frag/ip_frag_common.h | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/lib/librte_ip_frag/ip_frag_common.h b/lib/librte_ip_frag/ip_frag_common.h
index 197acf8d8..d7980a304 100644
--- a/lib/librte_ip_frag/ip_frag_common.h
+++ b/lib/librte_ip_frag/ip_frag_common.h
@@ -70,8 +70,9 @@ ip_frag_key_invalidate(struct ip_frag_key * key)
 
 /* compare two keys */
-static inline int
+static inline uint64_t
 ip_frag_key_cmp(const struct ip_frag_key * k1, const struct ip_frag_key * k2)
 {
-	uint32_t i, val;
+	uint32_t i;
+	uint64_t val;
 	val = k1->id ^ k2->id;
 	for (i = 0; i < k1->key_len; i++)
-- 
2.19.0

---
  Diff of the applied patch vs upstream commit (please double-check if non-empty:
---
--- -	2018-11-23 10:22:54.282066154 +0000
+++ 0001-ip_frag-fix-overflow-in-key-comparison.patch	2018-11-23 10:22:54.000000000 +0000
@@ -1,8 +1,10 @@
-From 8721e07478c6edc9565645047742f79c654cfd1b Mon Sep 17 00:00:00 2001
+From 362b57a19d3d606c23dd4ce198d608e34351b22f Mon Sep 17 00:00:00 2001
 From: Li Han <han.li1@zte.com.cn>
 Date: Wed, 1 Aug 2018 22:01:03 -0400
 Subject: [PATCH] ip_frag: fix overflow in key comparison
 
+[ upstream commit 8721e07478c6edc9565645047742f79c654cfd1b ]
+
 in struct ip_frag_key,src_dst[] type is uint64_t.
 but "val" which to store the calc restult ,type is uint32_t.
 we may lost high 32 bit key. and function return value is int,
@@ -15,10 +17,10 @@
  1 file changed, 3 insertions(+), 2 deletions(-)
 
 diff --git a/lib/librte_ip_frag/ip_frag_common.h b/lib/librte_ip_frag/ip_frag_common.h
-index 0fdcc7d0f..0f62e2e16 100644
+index 197acf8d8..d7980a304 100644
 --- a/lib/librte_ip_frag/ip_frag_common.h
 +++ b/lib/librte_ip_frag/ip_frag_common.h
-@@ -76,8 +76,9 @@ ip_frag_key_invalidate(struct ip_frag_key * key)
+@@ -70,8 +70,9 @@ ip_frag_key_invalidate(struct ip_frag_key * key)
  
  /* compare two keys */
 -static inline int

^ permalink raw reply	[flat|nested] 72+ messages in thread

end of thread, other threads:[~2018-11-26 15:42 UTC | newest]

Thread overview: 72+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-11-23 10:26 [dpdk-stable] patch 'ip_frag: fix overflow in key comparison' has been queued to stable release 18.08.1 Kevin Traynor
2018-11-23 10:26 ` [dpdk-stable] patch 'eal/linux: fix memory leak of logid' " Kevin Traynor
2018-11-23 10:26 ` [dpdk-stable] patch 'malloc: check size hint when reserving the biggest element' " Kevin Traynor
2018-11-23 10:26 ` [dpdk-stable] patch 'bus/vdev: fix devargs after multi-process bus scan' " Kevin Traynor
2018-11-23 10:26 ` [dpdk-stable] patch 'vfio: fix sPAPR IOMMU mapping' " Kevin Traynor
2018-11-23 10:26 ` [dpdk-stable] patch 'net/virtio: fix PCI config error handling' " Kevin Traynor
2018-11-26  2:02   ` Tiwei Bie
2018-11-26 15:42     ` Kevin Traynor
2018-11-23 10:26 ` [dpdk-stable] patch 'bus/pci: compare kernel driver instead of interrupt handler' " Kevin Traynor
2018-11-23 10:26 ` [dpdk-stable] patch 'vfio: check if group fd is already open' " Kevin Traynor
2018-11-23 10:26 ` [dpdk-stable] patch 'vfio: fix read of freed memory on getting container fd' " Kevin Traynor
2018-11-23 10:26 ` [dpdk-stable] patch 'vfio: share default container in multi-process' " Kevin Traynor
2018-11-23 10:26 ` [dpdk-stable] patch 'fix global variable issues' " Kevin Traynor
2018-11-23 10:26 ` [dpdk-stable] patch 'net/virtio: register/unregister intr handler on start/stop' " Kevin Traynor
2018-11-23 10:26 ` [dpdk-stable] patch 'eal/linux: handle UIO read failure in interrupt handler' " Kevin Traynor
2018-11-23 10:26 ` [dpdk-stable] patch 'examples/vm_power: respect maximum CPUs' " Kevin Traynor
2018-11-23 10:26 ` [dpdk-stable] patch 'mk: disable gcc AVX512F support' " Kevin Traynor
2018-11-23 10:26 ` [dpdk-stable] patch 'crypto/scheduler: fix build with gcc 8.2' " Kevin Traynor
2018-11-23 10:26 ` [dpdk-stable] patch 'bus/dpaa: fix build with gcc 9.0' " Kevin Traynor
2018-11-23 10:26 ` [dpdk-stable] patch 'eal: " Kevin Traynor
2018-11-23 10:26 ` [dpdk-stable] patch 'app/testpmd: fix shaper profile parameters' " Kevin Traynor
2018-11-23 10:26 ` [dpdk-stable] patch 'app/testpmd: fix RED byte stats' " Kevin Traynor
2018-11-23 10:26 ` [dpdk-stable] patch 'examples/ip_pipeline: fix port and table stats read' " Kevin Traynor
2018-11-23 10:26 ` [dpdk-stable] patch 'crypto/openssl: fix RSA verify operation' " Kevin Traynor
2018-11-23 10:26 ` [dpdk-stable] patch 'compressdev: clarify usage of op structure' " Kevin Traynor
2018-11-23 10:26 ` [dpdk-stable] patch 'compressdev: fix op allocation' " Kevin Traynor
2018-11-23 10:26 ` [dpdk-stable] patch 'compress/isal: fix uncleared compression states' " Kevin Traynor
2018-11-23 10:26 ` [dpdk-stable] patch 'ring/c11: synchronize load and store of the tail' " Kevin Traynor
2018-11-23 10:26 ` [dpdk-stable] patch 'ring/c11: move atomic load of head above the loop' " Kevin Traynor
2018-11-23 10:26 ` [dpdk-stable] patch 'net/i40e: fix offload not supported mask' " Kevin Traynor
2018-11-23 10:26 ` [dpdk-stable] patch 'net/mlx5: use pkg-config to handle SUSE libmnl' " Kevin Traynor
2018-11-23 10:26 ` [dpdk-stable] patch 'net/vhost: fix parameters string' " Kevin Traynor
2018-11-23 10:26 ` [dpdk-stable] patch 'net/virtio-user: do not stop stopped device again' " Kevin Traynor
2018-11-23 10:26 ` [dpdk-stable] patch 'net/virtio-user: do not make vhost channel non-block' " Kevin Traynor
2018-11-23 10:26 ` [dpdk-stable] patch 'net/virtio-user: do not reset owner when driver resets' " Kevin Traynor
2018-11-23 10:26 ` [dpdk-stable] patch 'net/virtio-user: fix device features for server mode' " Kevin Traynor
2018-11-23 10:26 ` [dpdk-stable] patch 'net/virtio: fix guest announce support' " Kevin Traynor
2018-11-23 10:26 ` [dpdk-stable] patch 'net: fix build with pedantic' " Kevin Traynor
2018-11-23 10:26 ` [dpdk-stable] patch 'net/ixgbe: fix busy polling while fiber link update' " Kevin Traynor
2018-11-23 10:26 ` [dpdk-stable] patch 'net/virtio: do not re-enter clean up routines' " Kevin Traynor
2018-11-23 10:26 ` [dpdk-stable] patch 'ethdev: fix redundant function pointer check' " Kevin Traynor
2018-11-23 10:26 ` [dpdk-stable] patch 'app/testpmd: fix Tx offload flags' " Kevin Traynor
2018-11-23 10:26 ` [dpdk-stable] patch 'net/bonding: fix crash on probe' " Kevin Traynor
2018-11-23 10:26 ` [dpdk-stable] patch 'doc: clarify TSO Tx offload prerequisite' " Kevin Traynor
2018-11-23 10:26 ` [dpdk-stable] patch 'vhost/crypto: fix inferred misuse of enum' " Kevin Traynor
2018-11-23 10:26 ` [dpdk-stable] patch 'net/igb: update Tx offload mask' " Kevin Traynor
2018-11-23 10:26 ` [dpdk-stable] patch 'net/avf/base: fix shifting 32-bit signed variable 31 times' " Kevin Traynor
2018-11-23 10:26 ` [dpdk-stable] patch 'net/i40e: fix Rx instability with vector mode' " Kevin Traynor
2018-11-23 10:26 ` [dpdk-stable] patch 'net/bnxt: fix uninitialized variable access' " Kevin Traynor
2018-11-23 10:26 ` [dpdk-stable] patch 'lib: fix shifting 32-bit signed variable 31 times' " Kevin Traynor
2018-11-23 10:26 ` [dpdk-stable] patch 'service: fix possible null access' " Kevin Traynor
2018-11-23 10:26 ` [dpdk-stable] patch 'vhost: fix possible out of bound " Kevin Traynor
2018-11-23 10:26 ` [dpdk-stable] patch 'ip_frag: check fragment length of incoming packet' " Kevin Traynor
2018-11-23 10:26 ` [dpdk-stable] patch 'bus/pci: fix config r/w access' " Kevin Traynor
2018-11-23 10:26 ` [dpdk-stable] patch 'examples/flow_filtering: filter out unsupported offloads' " Kevin Traynor
2018-11-23 10:26 ` [dpdk-stable] patch 'examples/flow_filtering: remove VLAN item' " Kevin Traynor
2018-11-23 10:26 ` [dpdk-stable] patch 'test/hash: fix build' " Kevin Traynor
2018-11-23 10:27 ` [dpdk-stable] patch 'eal/x86: remove unused memcpy file' " Kevin Traynor
2018-11-23 10:27 ` [dpdk-stable] patch 'test: fix build' " Kevin Traynor
2018-11-23 10:27 ` [dpdk-stable] patch 'ring/c11: keep deterministic order allowing retry to work' " Kevin Traynor
2018-11-23 10:27 ` [dpdk-stable] patch 'ring/c11: relax ordering for load and store of the head' " Kevin Traynor
2018-11-23 10:27 ` [dpdk-stable] patch 'pci: fix parsing of address without function number' " Kevin Traynor
2018-11-23 10:27 ` [dpdk-stable] patch 'bpf: fix x86 JIT for immediate loads' " Kevin Traynor
2018-11-23 10:27 ` [dpdk-stable] patch 'ipc: remove panic in async request' " Kevin Traynor
2018-11-23 10:27 ` [dpdk-stable] patch 'hash: fix TSX aborts with newer gcc' " Kevin Traynor
2018-11-23 10:27 ` [dpdk-stable] patch 'bus/vmbus: fix directory handle leak on error' " Kevin Traynor
2018-11-23 10:27 ` [dpdk-stable] patch 'net/tap: fix file descriptor " Kevin Traynor
2018-11-23 10:27 ` [dpdk-stable] patch 'net/tap: fix file descriptor check' " Kevin Traynor
2018-11-23 10:27 ` [dpdk-stable] patch 'examples/flow_filtering: fix capability setting' " Kevin Traynor
2018-11-23 10:27 ` [dpdk-stable] patch 'app/testpmd: fix Rx offload search' " Kevin Traynor
2018-11-23 10:27 ` [dpdk-stable] patch 'net/tap: fix probe for multiq or flowq failure' " Kevin Traynor
2018-11-23 11:02   ` Varghese, Vipin

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).