examples/pipeline: fix build
Checks
Commit Message
This patch fixes the following build failure seen on Ubuntu 16.04
with gcc 5.4.0 because of uninitialized variable:
[..]
examples/pipeline/cli.c:2853:9: error: 'session_id' may be used
uninitialized in this function [-Werror=maybe-uninitialized]
[..]
Fixes: 172254555f9f ("examples/pipeline: support packet mirroring")
Cc: cristian.dumitrescu@intel.com
Signed-off-by: Ali Alnubani <alialnu@nvidia.com>
---
examples/pipeline/cli.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
Comments
> -----Original Message-----
> From: Ali Alnubani <alialnu@nvidia.com>
> Sent: Thursday, June 2, 2022 1:34 PM
> To: dev@dpdk.org
> Cc: Dumitrescu, Cristian <cristian.dumitrescu@intel.com>
> Subject: [PATCH] examples/pipeline: fix build
>
> This patch fixes the following build failure seen on Ubuntu 16.04
> with gcc 5.4.0 because of uninitialized variable:
> [..]
> examples/pipeline/cli.c:2853:9: error: 'session_id' may be used
> uninitialized in this function [-Werror=maybe-uninitialized]
> [..]
>
> Fixes: 172254555f9f ("examples/pipeline: support packet mirroring")
> Cc: cristian.dumitrescu@intel.com
>
> Signed-off-by: Ali Alnubani <alialnu@nvidia.com>
> ---
Acked-by: Cristian Dumitrescu <cristian.dumitrescu@intel.com>
> > This patch fixes the following build failure seen on Ubuntu 16.04
> > with gcc 5.4.0 because of uninitialized variable:
> > [..]
> > examples/pipeline/cli.c:2853:9: error: 'session_id' may be used
> > uninitialized in this function [-Werror=maybe-uninitialized]
> > [..]
> >
> > Fixes: 172254555f9f ("examples/pipeline: support packet mirroring")
> > Cc: cristian.dumitrescu@intel.com
> >
> > Signed-off-by: Ali Alnubani <alialnu@nvidia.com>
>
> Acked-by: Cristian Dumitrescu <cristian.dumitrescu@intel.com>
Applied, thanks.
@@ -2782,7 +2782,7 @@ cmd_pipeline_mirror_session(char **tokens,
{
struct rte_swx_pipeline_mirroring_session_params params;
struct pipeline *p;
- uint32_t session_id;
+ uint32_t session_id = 0;
int status;
if (n_tokens != 11) {