Thanks for the reviews. I applied these to master.
On Wed, Oct 16, 2013 at 02:20:51PM -0700, Alex Wang wrote: > Looks good to me, > > > On Mon, Sep 30, 2013 at 2:47 PM, Ben Pfaff <b...@nicira.com> wrote: > > > Signed-off-by: Ben Pfaff <b...@nicira.com> > > --- > > lib/bfd.c | 6 ++++-- > > 1 file changed, 4 insertions(+), 2 deletions(-) > > > > diff --git a/lib/bfd.c b/lib/bfd.c > > index 6c9e920..dbb0e13 100644 > > --- a/lib/bfd.c > > +++ b/lib/bfd.c > > @@ -625,8 +625,10 @@ bfd_process_packet(struct bfd *bfd, const struct flow > > *flow, > > > > msg = ofpbuf_at(p, (uint8_t *)p->l7 - (uint8_t *)p->data, > > BFD_PACKET_LEN); > > if (!msg) { > > - VLOG_INFO_RL(&rl, "%s: Received unparseable BFD control message.", > > - bfd->name); > > + VLOG_INFO_RL(&rl, "%s: Received too-short BFD control message > > (only " > > + "%td bytes long, at least %d required).", > > + bfd->name, (uint8_t *) ofpbuf_tail(p) - (uint8_t *) > > p->l7, > > + BFD_PACKET_LEN); > > goto out; > > } > > > > -- > > 1.7.10.4 > > > > _______________________________________________ > > dev mailing list > > dev@openvswitch.org > > http://openvswitch.org/mailman/listinfo/dev > > _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev