diff options
author | Jarno Rajahalme <jarno.rajahalme@nsn.com> | 2012-11-30 13:45:08 -0800 |
---|---|---|
committer | Ben Pfaff <blp@nicira.com> | 2012-12-04 08:25:55 -0800 |
commit | 15549878535c7331c3952e656a2eb9944d0021bb (patch) | |
tree | d705f34ee1c00f3e4a7b4889e3d77aefe37dcf7a /lib/ofp-util.c | |
parent | 7a7b64289cd96c5019168f7f0905f59e2d078b76 (diff) |
Don't use error categories as if they were actually errors.
The OFPERR_* naming scheme is confusing, because it mixes categories with
specific errors. Categories can't be encoded for sending to controllers,
so this is a problem. This commit fixes up the cases where categories
were used as errors, replacing them by real errors types.
Signed-off-by: Jarno Rajahalme <jarno.rajahalme@nsn.com>
Signed-off-by: Ben Pfaff <blp@nicira.com>
Diffstat (limited to 'lib/ofp-util.c')
-rw-r--r-- | lib/ofp-util.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/ofp-util.c b/lib/ofp-util.c index 320bfde4..11b6192e 100644 --- a/lib/ofp-util.c +++ b/lib/ofp-util.c @@ -3681,7 +3681,7 @@ ofputil_decode_flow_update(struct ofputil_flow_update *update, VLOG_WARN_RL(&bad_ofmsg_rl, "NXST_FLOW_MONITOR reply has bad event %"PRIu16, ntohs(nfuh->event)); - return OFPERR_OFPET_BAD_REQUEST; + return OFPERR_NXBRC_FM_BAD_EVENT; } bad_len: |