Commit d1a23fa6 authored by Christophe Massiot's avatar Christophe Massiot

Fixed a side-effect bug with IPv6 addresses (thanks Xag).

parent 44eb44e9
...@@ -2,7 +2,7 @@ ...@@ -2,7 +2,7 @@
* ipv6.c: IPv6 network abstraction layer * ipv6.c: IPv6 network abstraction layer
***************************************************************************** *****************************************************************************
* Copyright (C) 2002 VideoLAN * Copyright (C) 2002 VideoLAN
* $Id: ipv6.c,v 1.5 2002/11/28 23:24:15 massiot Exp $ * $Id: ipv6.c,v 1.6 2002/12/23 16:05:04 massiot Exp $
* *
* Authors: Alexis Guillard <alexis.guillard@bt.com> * Authors: Alexis Guillard <alexis.guillard@bt.com>
* Christophe Massiot <massiot@via.ecp.fr> * Christophe Massiot <massiot@via.ecp.fr>
...@@ -85,9 +85,10 @@ vlc_module_end(); ...@@ -85,9 +85,10 @@ vlc_module_end();
* BuildAddr: utility function to build a struct sockaddr_in6 * BuildAddr: utility function to build a struct sockaddr_in6
*****************************************************************************/ *****************************************************************************/
static int BuildAddr( vlc_object_t * p_this, struct sockaddr_in6 * p_socket, static int BuildAddr( vlc_object_t * p_this, struct sockaddr_in6 * p_socket,
char * psz_address, int i_port ) const char * psz_bind_address, int i_port )
{ {
char * psz_multicast_interface = ""; char * psz_multicast_interface = "";
char * psz_address = strdup(psz_bind_address);
#if defined(WIN32) #if defined(WIN32)
/* Try to get getaddrinfo() and freeaddrinfo() from wship6.dll */ /* Try to get getaddrinfo() and freeaddrinfo() from wship6.dll */
...@@ -113,6 +114,7 @@ static int BuildAddr( vlc_object_t * p_this, struct sockaddr_in6 * p_socket, ...@@ -113,6 +114,7 @@ static int BuildAddr( vlc_object_t * p_this, struct sockaddr_in6 * p_socket,
{ {
msg_Err( p_this, "no IPv6 stack installed" ); msg_Err( p_this, "no IPv6 stack installed" );
if( wship6_dll ) FreeLibrary( wship6_dll ); if( wship6_dll ) FreeLibrary( wship6_dll );
free( psz_address );
return( -1 ); return( -1 );
} }
#endif #endif
...@@ -159,6 +161,7 @@ static int BuildAddr( vlc_object_t * p_this, struct sockaddr_in6 * p_socket, ...@@ -159,6 +161,7 @@ static int BuildAddr( vlc_object_t * p_this, struct sockaddr_in6 * p_socket,
if( _getaddrinfo( psz_address, NULL, &hints, &res ) ) if( _getaddrinfo( psz_address, NULL, &hints, &res ) )
{ {
FreeLibrary( wship6_dll ); FreeLibrary( wship6_dll );
free( psz_address );
return( -1 ); return( -1 );
} }
memcpy( &p_socket->sin6_addr, memcpy( &p_socket->sin6_addr,
...@@ -177,6 +180,7 @@ static int BuildAddr( vlc_object_t * p_this, struct sockaddr_in6 * p_socket, ...@@ -177,6 +180,7 @@ static int BuildAddr( vlc_object_t * p_this, struct sockaddr_in6 * p_socket,
if ( (p_hostent = gethostbyname2( psz_address, AF_INET6 )) == NULL ) if ( (p_hostent = gethostbyname2( psz_address, AF_INET6 )) == NULL )
{ {
msg_Err( p_this, "ipv6 error: unknown host %s", psz_address ); msg_Err( p_this, "ipv6 error: unknown host %s", psz_address );
free( psz_address );
return( -1 ); return( -1 );
} }
...@@ -188,6 +192,7 @@ static int BuildAddr( vlc_object_t * p_this, struct sockaddr_in6 * p_socket, ...@@ -188,6 +192,7 @@ static int BuildAddr( vlc_object_t * p_this, struct sockaddr_in6 * p_socket,
if( _getaddrinfo( psz_address, NULL, &hints, &res ) ) if( _getaddrinfo( psz_address, NULL, &hints, &res ) )
{ {
FreeLibrary( wship6_dll ); FreeLibrary( wship6_dll );
free( psz_address );
return( -1 ); return( -1 );
} }
memcpy( &p_socket->sin6_addr, memcpy( &p_socket->sin6_addr,
...@@ -198,6 +203,7 @@ static int BuildAddr( vlc_object_t * p_this, struct sockaddr_in6 * p_socket, ...@@ -198,6 +203,7 @@ static int BuildAddr( vlc_object_t * p_this, struct sockaddr_in6 * p_socket,
#else #else
msg_Err( p_this, "ipv6 error: IPv6 address %s is invalid", msg_Err( p_this, "ipv6 error: IPv6 address %s is invalid",
psz_address ); psz_address );
free( psz_address );
return( -1 ); return( -1 );
#endif #endif
} }
...@@ -206,7 +212,8 @@ static int BuildAddr( vlc_object_t * p_this, struct sockaddr_in6 * p_socket, ...@@ -206,7 +212,8 @@ static int BuildAddr( vlc_object_t * p_this, struct sockaddr_in6 * p_socket,
FreeLibrary( wship6_dll ); FreeLibrary( wship6_dll );
#endif #endif
return( 0 ); free( psz_address );
return 0;
} }
/***************************************************************************** /*****************************************************************************
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment