[ovs-dev] [PATCH 1/3] odp-util: Fix parsing of actions encapsulated within "sample" actions.

Ben Pfaff blp at nicira.com
Thu May 10 22:47:11 UTC 2012


Thanks, I pushed this patch to master and branch-1.[4567].

On Wed, May 09, 2012 at 01:39:55PM -0700, Ethan Jackson wrote:
> Looks good, thanks.
> 
> Ethan
> 
> On Wed, May 9, 2012 at 12:31 PM, Ben Pfaff <blp at nicira.com> wrote:
> > Signed-off-by: Ben Pfaff <blp at nicira.com>
> > ---
> >  lib/odp-util.c |    3 +--
> >  1 files changed, 1 insertions(+), 2 deletions(-)
> >
> > diff --git a/lib/odp-util.c b/lib/odp-util.c
> > index 88986a2..ce45635 100644
> > --- a/lib/odp-util.c
> > +++ b/lib/odp-util.c
> > @@ -534,7 +534,7 @@ parse_odp_action(const char *s, const struct shash *port_names,
> >             for (;;) {
> >                 int retval;
> >
> > -                s += strspn(s, delimiters);
> > +                n += strspn(s + n, delimiters);
> >                 if (s[n] == ')') {
> >                     break;
> >                 }
> > @@ -544,7 +544,6 @@ parse_odp_action(const char *s, const struct shash *port_names,
> >                     return retval;
> >                 }
> >                 n += retval;
> > -
> >             }
> >             nl_msg_end_nested(actions, actions_ofs);
> >             nl_msg_end_nested(actions, sample_ofs);
> > --
> > 1.7.2.5
> >
> > _______________________________________________
> > dev mailing list
> > dev at openvswitch.org
> > http://openvswitch.org/mailman/listinfo/dev



More information about the dev mailing list