actions (in xlate_actions__) would be NULL when xlate_actions() is called from packet_out()->ofproto_dpif_execute_actions(). This causes a NULL pointer to be dereferenced when ctx.xbridge->netflow is set.
Signed-off-by: Anoob Soman <[email protected]> --- ofproto/ofproto-dpif-xlate.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/ofproto/ofproto-dpif-xlate.c b/ofproto/ofproto-dpif-xlate.c index da538b7..f08d1ec 100644 --- a/ofproto/ofproto-dpif-xlate.c +++ b/ofproto/ofproto-dpif-xlate.c @@ -3417,9 +3417,6 @@ xlate_actions__(struct xlate_in *xin, struct xlate_out *xout) } if (ctx.xbridge->netflow) { - const struct ofpact *ofpacts = actions->ofpacts; - size_t ofpacts_len = actions->ofpacts_len; - /* Only update netflow if we don't have controller flow. We don't * report NetFlow expiration messages for such facets because they * are just part of the control logic for the network, not real -- 1.9.1 _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
