examples/pipeline: fix build
Checks
Commit Message
This patch fixes the following build failures seen on Ubuntu 16.04
with gcc 5.4.0 because of uninitialized variables:
...
examples/pipeline/cli.c:1559:11: error: 'weight_val' may be used
uninitialized in this function [-Werror=maybe-uninitialized]
...
examples/pipeline/cli.c:1545:13: error: 'member_id_val' may be used
uninitialized in this function [-Werror=maybe-uninitialized]
...
examples/pipeline/cli.c:1538:12: error: 'group_id_val' may be used
uninitialized in this function [-Werror=maybe-uninitialized]
...
examples/pipeline/cli.c:2189:2: error: 'idx1' may be used
uninitialized in this function [-Werror=maybe-uninitialized]
...
examples/pipeline/cli.c:2179:43: error: 'idx0' may be used
uninitialized in this function [-Werror=maybe-uninitialized]
...
examples/pipeline/cli.c:2265:2: error: 'idx1' may be used
uninitialized in this function [-Werror=maybe-uninitialized]
...
examples/pipeline/cli.c:2248:43: error: 'idx0' may be used
uninitialized in this function [-Werror=maybe-uninitialized]
...
examples/pipeline/cli.c:2358:2: error: 'idx1' may be used
uninitialized in this function [-Werror=maybe-uninitialized]
...
examples/pipeline/cli.c:2325:43: error: 'idx0' may be used
uninitialized in this function [-Werror=maybe-uninitialized]
Fixes: 598fe0dd0d8e ("examples/pipeline: support selector table")
Cc: cristian.dumitrescu@intel.com
Signed-off-by: Ali Alnubani <alialnu@nvidia.com>
---
examples/pipeline/cli.c | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
Comments
> -----Original Message-----
> From: Ali Alnubani <alialnu@nvidia.com>
> Sent: Monday, July 12, 2021 8:47 AM
> To: dev@dpdk.org
> Cc: thomas@monjalon.net; Khangar, Churchill
> <churchill.khangar@intel.com>; Dumitrescu, Cristian
> <cristian.dumitrescu@intel.com>
> Subject: [PATCH] examples/pipeline: fix build
>
> This patch fixes the following build failures seen on Ubuntu 16.04
> with gcc 5.4.0 because of uninitialized variables:
> ...
> examples/pipeline/cli.c:1559:11: error: 'weight_val' may be used
> uninitialized in this function [-Werror=maybe-uninitialized]
> ...
> examples/pipeline/cli.c:1545:13: error: 'member_id_val' may be used
> uninitialized in this function [-Werror=maybe-uninitialized]
> ...
> examples/pipeline/cli.c:1538:12: error: 'group_id_val' may be used
> uninitialized in this function [-Werror=maybe-uninitialized]
> ...
> examples/pipeline/cli.c:2189:2: error: 'idx1' may be used
> uninitialized in this function [-Werror=maybe-uninitialized]
> ...
> examples/pipeline/cli.c:2179:43: error: 'idx0' may be used
> uninitialized in this function [-Werror=maybe-uninitialized]
> ...
> examples/pipeline/cli.c:2265:2: error: 'idx1' may be used
> uninitialized in this function [-Werror=maybe-uninitialized]
> ...
> examples/pipeline/cli.c:2248:43: error: 'idx0' may be used
> uninitialized in this function [-Werror=maybe-uninitialized]
> ...
> examples/pipeline/cli.c:2358:2: error: 'idx1' may be used
> uninitialized in this function [-Werror=maybe-uninitialized]
> ...
> examples/pipeline/cli.c:2325:43: error: 'idx0' may be used
> uninitialized in this function [-Werror=maybe-uninitialized]
>
> Fixes: 598fe0dd0d8e ("examples/pipeline: support selector table")
> Cc: cristian.dumitrescu@intel.com
>
> Signed-off-by: Ali Alnubani <alialnu@nvidia.com>
> ---
> examples/pipeline/cli.c | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
Acked-by: Cristian Dumitrescu <cristian.dumitrescu@intel.com>
12/07/2021 10:08, Dumitrescu, Cristian:
> From: Ali Alnubani <alialnu@nvidia.com>
> > This patch fixes the following build failures seen on Ubuntu 16.04
> > with gcc 5.4.0 because of uninitialized variables:
>
> Acked-by: Cristian Dumitrescu <cristian.dumitrescu@intel.com>
Applied, thanks
@@ -1491,7 +1491,7 @@ pipeline_selector_group_member_read(const char *string,
{
char *token_array[GROUP_MEMBER_INFO_TOKENS_MAX], **tokens;
char *s0 = NULL, *s;
- uint32_t n_tokens = 0, group_id_val, member_id_val, weight_val;
+ uint32_t n_tokens = 0, group_id_val = 0, member_id_val = 0, weight_val = 0;
int blank_or_comment = 0;
/* Check input arguments. */
@@ -2141,7 +2141,7 @@ cmd_pipeline_meter_reset(char **tokens,
{
struct pipeline *p;
const char *name;
- uint32_t idx0, idx1;
+ uint32_t idx0 = 0, idx1 = 0;
if (n_tokens != 9) {
snprintf(out, out_size, MSG_ARG_MISMATCH, tokens[0]);
@@ -2210,7 +2210,7 @@ cmd_pipeline_meter_set(char **tokens,
{
struct pipeline *p;
const char *name, *profile_name;
- uint32_t idx0, idx1;
+ uint32_t idx0 = 0, idx1 = 0;
if (n_tokens != 11) {
snprintf(out, out_size, MSG_ARG_MISMATCH, tokens[0]);
@@ -2287,7 +2287,7 @@ cmd_pipeline_meter_stats(char **tokens,
struct rte_swx_ctl_meter_stats stats;
struct pipeline *p;
const char *name;
- uint32_t idx0, idx1;
+ uint32_t idx0 = 0, idx1 = 0;
if (n_tokens != 9) {
snprintf(out, out_size, MSG_ARG_MISMATCH, tokens[0]);