Looks good.
Ethan
On Wed, Nov 2, 2011 at 12:59, Ben Pfaff <b...@nicira.com> 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." <vora...@kth.se>
> Reported-by: Jari Sundell <sundell.softw...@gmail.com>
> ---
> 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 hen...@nicira.com
> Jad Naous jna...@gmail.com
> Jan Medved jmed...@juniper.net
> Janis Hamme janis.ha...@student.kit.edu
> +Jari Sundell sundell.softw...@gmail.com
> Jed Daniels openvswi...@jeddaniels.com
> Jeongkeun Lee jk...@hp.com
> Joan Cirer j...@ev0.net
> @@ -105,6 +106,7 @@ Takayuki HAMA t-h...@cb.jp.nec.com
> Teemu Koponen kopo...@nicira.com
> Tyler Coumbes coum...@gmail.com
> Vishal Swarankar vishal.swarn...@gmail.com
> +Voravit T. vora...@kth.se
> YAMAMOTO Takashi yamam...@valinux.co.jp
> Yongqiang Liu liuyq7...@gmail.com
> kk yap yap...@stanford.edu
> 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
> dev@openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev
>
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
