After more testing it turns out it isn't the patch itself, rather something that was committed since 65c3058c22.
Jari Sundell On Fri, Nov 4, 2011 at 4:30 PM, Jari Sundell <[email protected]> wrote: > Seems things are not yet done... Without this patch (git master) > packets are properly being passed to local port when disable-in-band > is true. > > With the patch no packets get through, with or without disable-in-band set. > > Jari Sundell > > On Thu, Nov 3, 2011 at 5:10 AM, Ben Pfaff <[email protected]> wrote: >> Thanks, pushed to master, branch-1.3, branch-1.2. >> >> On Wed, Nov 02, 2011 at 01:03:56PM -0700, Ethan Jackson wrote: >>> Looks good. >>> >>> Ethan >>> >>> On Wed, Nov 2, 2011 at 12:59, Ben Pfaff <[email protected]> wrote: >>> > stream_init() didn't initialize the remote_ip, remote_port, local_ip, or >>> > local_port members of the stream, so "unix" streams that don't have any of >>> > those would get random values instead. >>> > >>> > Reported-by: "Voravit T." <[email protected]> >>> > Reported-by: Jari Sundell <[email protected]> >>> > --- >>> > ?AUTHORS ? ? ?| ? ?2 ++ >>> > ?lib/stream.c | ? ?1 + >>> > ?2 files changed, 3 insertions(+), 0 deletions(-) >>> > >>> > diff --git a/AUTHORS b/AUTHORS >>> > index d19d665..0d4cc95 100644 >>> > --- a/AUTHORS >>> > +++ b/AUTHORS >>> > @@ -77,6 +77,7 @@ Henrik Amren ? ? ? ? ? [email protected] >>> > ?Jad Naous ? ? ? ? ? ? ? [email protected] >>> > ?Jan Medved ? ? ? ? ? ? [email protected] >>> > ?Janis Hamme ? ? ? ? ? ? [email protected] >>> > +Jari Sundell ? ? ? ? ? [email protected] >>> > ?Jed Daniels ? ? ? ? ? ? [email protected] >>> > ?Jeongkeun Lee ? ? ? ? ? [email protected] >>> > ?Joan Cirer ? ? ? ? ? ? [email protected] >>> > @@ -105,6 +106,7 @@ Takayuki HAMA ? ? ? ? ? [email protected] >>> > ?Teemu Koponen ? ? ? ? ? [email protected] >>> > ?Tyler Coumbes ? ? ? ? ? [email protected] >>> > ?Vishal Swarankar ? ? ? [email protected] >>> > +Voravit T. ? ? ? ? ? ? [email protected] >>> > ?YAMAMOTO Takashi ? ? ? [email protected] >>> > ?Yongqiang Liu ? ? ? ? ? [email protected] >>> > ?kk yap ? ? ? ? ? ? ? ? [email protected] >>> > diff --git a/lib/stream.c b/lib/stream.c >>> > index 37b6110..8f567ca 100644 >>> > --- a/lib/stream.c >>> > +++ b/lib/stream.c >>> > @@ -616,6 +616,7 @@ void >>> > ?stream_init(struct stream *stream, struct stream_class *class, >>> > ? ? ? ? ? ? int connect_status, const char *name) >>> > ?{ >>> > + ? ?memset(stream, 0, sizeof *stream); >>> > ? ? stream->class = class; >>> > ? ? stream->state = (connect_status == EAGAIN ? SCS_CONNECTING >>> > ? ? ? ? ? ? ? ? ? ? : !connect_status ? SCS_CONNECTED >>> > -- >>> > 1.7.4.4 >>> > >>> > _______________________________________________ >>> > dev mailing list >>> > [email protected] >>> > http://openvswitch.org/mailman/listinfo/dev >>> > >> > _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
