From fe4b8c7bcdba8c81b77e5c28b0b55c5a6653a3c6 Mon Sep 17 00:00:00 2001 From: Ali Alnubani Date: Wed, 21 Apr 2021 13:06:38 +0300 Subject: [PATCH] pipeline: fix build with GCC 4.8.5 Compilation on CentOS 7 with gcc version 4.8.5 fails with the following errors: error: 'src_struct_id' may be used uninitialized in this function [-Werror=maybe-uninitialized] error: 'dst_struct_id' may be used uninitialized in this function [-Werror=maybe-uninitialized] This patch fixes the build errors by initializing both variables. Bugzilla ID: 683 Fixes: 783768136f29 ("pipeline: auto-detect endianness of action arguments") Signed-off-by: Ali Alnubani --- lib/librte_pipeline/rte_swx_pipeline.c | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/lib/librte_pipeline/rte_swx_pipeline.c b/lib/librte_pipeline/rte_swx_pipeline.c index 4e358bbda4..a2732a1e57 100644 --- a/lib/librte_pipeline/rte_swx_pipeline.c +++ b/lib/librte_pipeline/rte_swx_pipeline.c @@ -3624,7 +3624,7 @@ instr_mov_translate(struct rte_swx_pipeline *p, char *dst = tokens[1], *src = tokens[2]; struct field *fdst, *fsrc; uint64_t src_val; - uint32_t dst_struct_id, src_struct_id; + uint32_t dst_struct_id = 0, src_struct_id = 0; CHECK(n_tokens == 3, EINVAL); @@ -3889,7 +3889,7 @@ instr_alu_add_translate(struct rte_swx_pipeline *p, char *dst = tokens[1], *src = tokens[2]; struct field *fdst, *fsrc; uint64_t src_val; - uint32_t dst_struct_id, src_struct_id; + uint32_t dst_struct_id = 0, src_struct_id = 0; CHECK(n_tokens == 3, EINVAL); @@ -3942,7 +3942,7 @@ instr_alu_sub_translate(struct rte_swx_pipeline *p, char *dst = tokens[1], *src = tokens[2]; struct field *fdst, *fsrc; uint64_t src_val; - uint32_t dst_struct_id, src_struct_id; + uint32_t dst_struct_id = 0, src_struct_id = 0; CHECK(n_tokens == 3, EINVAL); @@ -4072,7 +4072,7 @@ instr_alu_shl_translate(struct rte_swx_pipeline *p, char *dst = tokens[1], *src = tokens[2]; struct field *fdst, *fsrc; uint64_t src_val; - uint32_t dst_struct_id, src_struct_id; + uint32_t dst_struct_id = 0, src_struct_id = 0; CHECK(n_tokens == 3, EINVAL); @@ -4125,7 +4125,7 @@ instr_alu_shr_translate(struct rte_swx_pipeline *p, char *dst = tokens[1], *src = tokens[2]; struct field *fdst, *fsrc; uint64_t src_val; - uint32_t dst_struct_id, src_struct_id; + uint32_t dst_struct_id = 0, src_struct_id = 0; CHECK(n_tokens == 3, EINVAL); @@ -4178,7 +4178,7 @@ instr_alu_and_translate(struct rte_swx_pipeline *p, char *dst = tokens[1], *src = tokens[2]; struct field *fdst, *fsrc; uint64_t src_val; - uint32_t dst_struct_id, src_struct_id; + uint32_t dst_struct_id = 0, src_struct_id = 0; CHECK(n_tokens == 3, EINVAL); @@ -4231,7 +4231,7 @@ instr_alu_or_translate(struct rte_swx_pipeline *p, char *dst = tokens[1], *src = tokens[2]; struct field *fdst, *fsrc; uint64_t src_val; - uint32_t dst_struct_id, src_struct_id; + uint32_t dst_struct_id = 0, src_struct_id = 0; CHECK(n_tokens == 3, EINVAL); @@ -4284,7 +4284,7 @@ instr_alu_xor_translate(struct rte_swx_pipeline *p, char *dst = tokens[1], *src = tokens[2]; struct field *fdst, *fsrc; uint64_t src_val; - uint32_t dst_struct_id, src_struct_id; + uint32_t dst_struct_id = 0, src_struct_id = 0; CHECK(n_tokens == 3, EINVAL); -- 2.20.1