diff options
author | Ethan Jackson <ethan@nicira.com> | 2013-01-23 17:05:54 -0800 |
---|---|---|
committer | Ethan Jackson <ethan@nicira.com> | 2013-01-28 13:15:46 -0800 |
commit | 72d64e3322bba929e733886fef50711558016701 (patch) | |
tree | 7b0cf333636a75fc834d0241a97338b4caaef796 /ofproto | |
parent | 58c6adda89411391d076ee45e503c2788c61ebf5 (diff) |
ofproto-dpif: Require an in_port when tracing datapath flows.
All datapath flows should have an in_port, so it doesn't make a lot
of sense to allow omitting it when tracing. If a user wants to
trace a flow which has no in_port, they can use the OpenFlow syntax
which doesn't go through ofproto_receive().
Signed-off-by: Ethan Jackson <ethan@nicira.com>
Diffstat (limited to 'ofproto')
-rw-r--r-- | ofproto/ofproto-dpif.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/ofproto/ofproto-dpif.c b/ofproto/ofproto-dpif.c index 10860366..0fdcaa7d 100644 --- a/ofproto/ofproto-dpif.c +++ b/ofproto/ofproto-dpif.c @@ -3526,7 +3526,7 @@ ofproto_receive(const struct dpif_backer *backer, struct ofpbuf *packet, port = odp_port_to_ofport(backer, flow->in_port); if (!port) { flow->in_port = OFPP_NONE; - error = ofproto ? ENODEV : 0; + error = ENODEV; goto exit; } |