diff options
| author | Ben Pfaff <blp@ovn.org> | 2018-04-11 11:24:59 -0700 |
|---|---|---|
| committer | Ben Pfaff <blp@ovn.org> | 2018-04-16 14:51:09 -0700 |
| commit | 1bb011218d046b5de765658e9f5fe45289462eea (patch) | |
| tree | 01d9f25a3032d3e7f7f235cf1500c4a3189bf213 /ofproto/netflow.c | |
| parent | 3a974b92c28cd66c0e1e9fd4b53c4c3b8fe722cb (diff) | |
| download | openvswitch-1bb011218d046b5de765658e9f5fe45289462eea.tar.gz | |
socket-util: Make inet_parse_active() and inet_parse_passive() more alike.
Until now, the default_port parameters to these functions have had
different types and different behavior. There is a reason for this, since
it makes sense to listen on a kernel-selected port but it does not make
sense to connect to a kernel-selected port, but this overlooks the
possibility that a caller might want to parse a string in the format
understood by inet_parse_active() without actually using it to connect to
a remote host. This commit makes the behavior consistent and updates all
the callers to work with the new semantics.
Signed-off-by: Ben Pfaff <blp@ovn.org>
Acked-by: Mark Michelson <mmichels@redhat.com>
Diffstat (limited to 'ofproto/netflow.c')
| -rw-r--r-- | ofproto/netflow.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/ofproto/netflow.c b/ofproto/netflow.c index b6baeb826..ed58de17d 100644 --- a/ofproto/netflow.c +++ b/ofproto/netflow.c @@ -362,7 +362,7 @@ netflow_set_options(struct netflow *nf, nf->add_id_to_iface = nf_options->add_id_to_iface; collectors_destroy(nf->collectors); - collectors_create(&nf_options->collectors, 0, &nf->collectors); + collectors_create(&nf_options->collectors, -1, &nf->collectors); old_timeout = nf->active_timeout; if (nf_options->active_timeout >= 0) { |