ofp-util: Fix OF1.4+ version of ofputil_decode_set_async_config().
authorBen Pfaff <blp@ovn.org>
Tue, 19 Jan 2016 00:00:34 +0000 (16:00 -0800)
committerBen Pfaff <blp@ovn.org>
Wed, 20 Jan 2016 17:48:59 +0000 (09:48 -0800)
commit5876b4dbf6887a9c6c419c130a8ef9996ac52012
treeeebf6ea6998889d5ae23d855021fbdb395e14268
parent8fd0bb602e49bff8b5b6db594083572bb5604025
ofp-util: Fix OF1.4+ version of ofputil_decode_set_async_config().

The OF1.0 through OF1.3 "set async config" set the whole configuration,
OF1.4+ only update parts of it piecemeal, but the decoding function always
set the whole configuration.  This commit fixes the problem by changing the
interface to require the caller to provide an initial state.  (It would
be possible to simply make it mutate existing state in-place, but that
interface seems a little more error-prone.)

Found by inspection.

Signed-off-by: Ben Pfaff <blp@ovn.org>
Acked-by: Jarno Rajahalme <jarno@ovn.org>
OPENFLOW-1.1+.md
lib/ofp-print.c
lib/ofp-util.c
lib/ofp-util.h
ofproto/ofproto.c