[PATCH] stack: occasional crash due to uninitialized variable
Andre Muezerie
andremue at linux.microsoft.com
Tue Feb 4 02:23:37 CET 2025
Variable "success" was not being initialized and there was a code
path where the last do/while loop in __rte_stack_lf_pop_elems
looked at the value of this variable before it was set to any.
Compiling with msvc resulted in stack_lf_autotest sometimes crashing.
The fix is to initialize the variable. This same fix was applied to
stack\rte_stack_lf_generic.h in the past but was missed here.
Fixes: 7e6e609939a8 ("stack: add C11 atomic implementation")
Cc: stable at dpdk.org
Signed-off-by: Andre Muezerie <andremue at linux.microsoft.com>
---
lib/stack/rte_stack_lf_c11.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/lib/stack/rte_stack_lf_c11.h b/lib/stack/rte_stack_lf_c11.h
index 60d46e963b..ee1c79c095 100644
--- a/lib/stack/rte_stack_lf_c11.h
+++ b/lib/stack/rte_stack_lf_c11.h
@@ -77,7 +77,7 @@ __rte_stack_lf_pop_elems(struct rte_stack_lf_list *list,
{
struct rte_stack_lf_head old_head;
uint64_t len;
- int success;
+ int success = 0;
/* Reserve num elements, if available */
len = rte_atomic_load_explicit(&list->len, rte_memory_order_relaxed);
--
2.47.2.vfs.0.1
More information about the stable
mailing list