[ovs-dev] [PATCH 13/13] ofproto-dpif-xlate: Fix dead assignment reported by clang.

Greg Rose gvrose8192 at gmail.com
Fri Sep 15 17:42:34 UTC 2017


On 09/08/2017 10:59 AM, Bhanuprakash Bodireddy wrote:
> Clang reports that value stored in to ac_offset is never read in the
> function.
> 
> Signed-off-by: Bhanuprakash Bodireddy <bhanuprakash.bodireddy at intel.com>
> ---
>   ofproto/ofproto-dpif-xlate.c | 1 -
>   1 file changed, 1 deletion(-)
> 
> diff --git a/ofproto/ofproto-dpif-xlate.c b/ofproto/ofproto-dpif-xlate.c
> index 9e1f837..5d83666 100644
> --- a/ofproto/ofproto-dpif-xlate.c
> +++ b/ofproto/ofproto-dpif-xlate.c
> @@ -5370,7 +5370,6 @@ compose_clone(struct xlate_ctx *ctx, const struct ofpact_nest *oc)
>       if (ctx->xbridge->support.clone) { /* Use clone action */
>           /* Use clone action as datapath clone. */
>           offset = nl_msg_start_nested(ctx->odp_actions, OVS_ACTION_ATTR_CLONE);
> -        ac_offset = ctx->odp_actions->size;
>           do_xlate_actions(oc->actions, oc_actions_len, ctx);
>           nl_msg_end_non_empty_nested(ctx->odp_actions, offset);
>           goto dp_clone_done;
> 

Not in this section of code anyway.

Looks good.

Reviewed-by: Greg Rose <gvrose8192 at gmail.com>


More information about the dev mailing list