On Tue, Dec 15, 2015 at 10:51 PM, Ben Pfaff <[email protected]> wrote: > Reported-by: weizj <[email protected]> > Reported-at: https://github.com/openvswitch/ovs/pull/97 > Signed-off-by: Ben Pfaff <[email protected]>
This fix makes sense to me. Acked-by: Andy Zhou <[email protected]> If all assignments to 'reply' were grouped together, it may be slightly easier to read. > --- > AUTHORS | 1 + > lib/ofp-util.c | 1 + > 2 files changed, 2 insertions(+) > > diff --git a/AUTHORS b/AUTHORS > index c5c9d51..cc1673c 100644 > --- a/AUTHORS > +++ b/AUTHORS > @@ -413,6 +413,7 @@ kk yap [email protected] > likunyun [email protected] > neeraj mehta [email protected] > rahim entezari [email protected] > +weizj [email protected] > 俊 赵 [email protected] > 冯全树(Crab) [email protected] > 胡靖飞 [email protected] > diff --git a/lib/ofp-util.c b/lib/ofp-util.c > index d7e9ccc..31f9e93 100644 > --- a/lib/ofp-util.c > +++ b/lib/ofp-util.c > @@ -2039,6 +2039,7 @@ ofputil_append_meter_config(struct ovs_list *replies, > > ofputil_put_bands(mc->n_bands, mc->bands, msg); > > + reply = ofpbuf_at_assert(msg, start_ofs, sizeof *reply); > reply->length = htons(msg->size - start_ofs); > > ofpmp_postappend(replies, start_ofs); > -- > 2.1.3 > > _______________________________________________ > dev mailing list > [email protected] > http://openvswitch.org/mailman/listinfo/dev _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
