On Mon, Oct 30, 2017 at 12:21:11PM +0100, Juan Quintela wrote:
> Signed-off-by: Juan Quintela <quint...@redhat.com>
> ---
> migration/migration.c | 25 ++++++++++++++++++-------
> migration/migration.h | 2 ++
> migration/socket.c | 7 +++++++
> 3 files changed, 27 insertions(+), 7 deletions(-)
> diff --git a/migration/socket.c b/migration/socket.c
> index 3a8232dd2d..c3ab81d1fb 100644
> --- a/migration/socket.c
> +++ b/migration/socket.c
> @@ -187,7 +187,14 @@ void tcp_start_incoming_migration(const char *host_port,
> Error **errp)
> Error *err = NULL;
> SocketAddress *saddr = tcp_build_address(host_port, &err);
> if (!err) {
> + char *new_uri;
> socket_start_incoming_migration(saddr, &err);
> + if (!err) {
> + new_uri = g_strdup_printf("tcp:%s:%s", saddr->u.inet.host,
> + saddr->u.inet.port);
This is bad as it is throwing away data that the original URI had. In particular
you loose the 'ipv4=on|off' and 'ipv6=on|off' flags. If you need to keep the
original URI for later, then why not just keep the 'host_port' parameter that
was passed into this function instead of trying to reverse engineeer the URI ?
Regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
