[ovs-dev] [PATCH 2/4] ofp-actions: Fix dead assignment.

William Tu u9012063 at gmail.com
Mon Oct 30 17:27:53 UTC 2017


Clang reports value stored to 'eah' in never read.
Fix it by removing it.

Signed-off-by: William Tu <u9012063 at gmail.com>
---
 lib/ofp-actions.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/ofp-actions.c b/lib/ofp-actions.c
index 71eb70c3c239..79041c281ed0 100644
--- a/lib/ofp-actions.c
+++ b/lib/ofp-actions.c
@@ -5535,7 +5535,7 @@ encode_CLONE(const struct ofpact_nest *clone,
     const size_t ofs = out->size;
     struct ext_action_header *eah;
 
-    eah = put_NXAST_CLONE(out);
+    put_NXAST_CLONE(out);
     len = ofpacts_put_openflow_actions(clone->actions,
                                        ofpact_nest_get_action_len(clone),
                                        out, ofp_version);
-- 
2.7.4



More information about the dev mailing list