Commit 918eef52 authored by Rémi Denis-Courmont's avatar Rémi Denis-Courmont

IPv6 support for HTTPd

parent fa2921ea
...@@ -5,7 +5,7 @@ ...@@ -5,7 +5,7 @@
* $Id$ * $Id$
* *
* Authors: Laurent Aimar <fenrir@via.ecp.fr> * Authors: Laurent Aimar <fenrir@via.ecp.fr>
* Remi Denis-Courmont <courmisch@via.ecp.fr> * Remi Denis-Courmont <courmisch@via.ecp.fr>
* *
* This program is free software; you can redistribute it and/or modify * This program is free software; you can redistribute it and/or modify
* it under the terms of the GNU General Public License as published by * it under the terms of the GNU General Public License as published by
...@@ -26,6 +26,7 @@ ...@@ -26,6 +26,7 @@
#include <vlc/vlc.h> #include <vlc/vlc.h>
#include "vlc_httpd.h" #include "vlc_httpd.h"
#include "network.h"
#include <string.h> #include <string.h>
#include <errno.h> #include <errno.h>
...@@ -39,9 +40,6 @@ ...@@ -39,9 +40,6 @@
#elif defined( WIN32 ) #elif defined( WIN32 )
# include <winsock2.h> # include <winsock2.h>
# include <ws2tcpip.h> # include <ws2tcpip.h>
# ifndef IN_MULTICAST
# define IN_MULTICAST(a) IN_CLASSD(a)
# endif
#else #else
# include <netdb.h> /* hostent ... */ # include <netdb.h> /* hostent ... */
# include <sys/socket.h> # include <sys/socket.h>
...@@ -854,33 +852,113 @@ void httpd_StreamDelete( httpd_stream_t *stream ) ...@@ -854,33 +852,113 @@ void httpd_StreamDelete( httpd_stream_t *stream )
*****************************************************************************/ *****************************************************************************/
#define LISTEN_BACKLOG 100 #define LISTEN_BACKLOG 100
#if defined( WIN32 ) || defined( UNDER_CE ) #if defined(HAVE_GETNAMEINFO) && !defined(HAVE_GETADDRINFO)
#define SOCKET_CLOSE(a) closesocket(a) /*
#else * For now, VLC's configure script does not check for getaddrinfo(),
#define SOCKET_CLOSE(a) close(a) * but it should be present if getnameinfo() is (the opposite is untrue, with
* Debian potato as an example)
*/
# define HAVE_GETADDRINFO 1
#endif #endif
static void httpd_HostThread( httpd_host_t * ); static void httpd_HostThread( httpd_host_t * );
static int BuildAddr( struct sockaddr_storage * p_socket, int *pi_sock_size, static int GetAddrPort( const struct sockaddr_storage *p_ss );
#ifndef HAVE_GETADDRINFO
struct httpd_addrinfo
{
int ai_family;
int ai_socktype;
int ai_protocol;
/*int ai_flags;*/
struct sockaddr *ai_addr;
int ai_addrlen;
struct httpd_addrinfo *ai_next;
};
# define addrinfo httpd_addrinfo
static int BuildAddr( struct sockaddr_in * p_socket,
const char * psz_address, int i_port ); const char * psz_address, int i_port );
#endif
/* create a new host */ /* create a new host */
httpd_host_t *httpd_HostNew( vlc_object_t *p_this, char *psz_host, int i_port ) httpd_host_t *httpd_HostNew( vlc_object_t *p_this, char *psz_host, int i_port )
{ {
httpd_t *httpd; httpd_t *httpd;
httpd_host_t *host; httpd_host_t *host = NULL;
vlc_value_t lockval; vlc_value_t lockval;
struct sockaddr_storage sock; int fd = -1;
int i, i_sock_size; struct addrinfo *res, *ptr;
/* resolv */ /* resolv */
if( BuildAddr( &sock, &i_sock_size, psz_host, i_port ) ) #ifdef HAVE_GETADDRINFO
{
vlc_value_t val;
char psz_port[6];
struct addrinfo hints;
memset( &hints, 0, sizeof( hints ) );
#if 0
/* Check if we have force ipv4 or ipv6 */
var_Create( p_this, "ipv4", VLC_VAR_BOOL | VLC_VAR_DOINHERIT );
var_Get( p_this, "ipv4", &val );
if( val.b_bool )
hints.ai_family = PF_INET;
#else
/*
* For now, keep IPv4 by default. That said, it should be safe to use
* IPv6 by default *on the server side*, as, apart from NetBSD, most
* systems accept IPv4 clients on IPv6 listening sockets.
*
*
*/
hints.ai_family = PF_INET;
#endif
var_Create( p_this, "ipv6", VLC_VAR_BOOL | VLC_VAR_DOINHERIT );
var_Get( p_this, "ipv6", &val );
if( val.b_bool )
hints.ai_family = PF_INET6;
hints.ai_socktype = SOCK_STREAM;
hints.ai_flags = AI_PASSIVE;
if (*psz_host == '\0')
psz_host = NULL;
snprintf( psz_port, sizeof( psz_port ), "%d", i_port );
psz_port[sizeof( psz_port ) - 1] = '\0';
if( getaddrinfo( psz_host, psz_port, &hints, &res ) )
{
msg_Err( p_this, "cannot resolve %s:%d", psz_host, i_port );
return NULL;
}
}
#else
struct sockaddr_in sock;
struct httpd_addrinfo info;
info.ai_family = PF_INET;
info.ai_socktype = SOCK_STREAM;
info.ai_protocol = 0;
info.ai_addr = (struct sockaddr *)&sock;
info.ai_addrlen = sizeof( sock );
info.ai_next = NULL;
res = &info;
if( BuildAddr( &sock, psz_host, i_port ) )
{ {
msg_Err( p_this, "cannot build address for %s:%d", psz_host, i_port ); msg_Err( p_this, "cannot build address for %s:%d", psz_host, i_port );
return NULL; return NULL;
} }
# define freeaddrinfo( r ) (void)0;
#endif
/* to be sure to avoid multiple creation */ /* to be sure to avoid multiple creation */
var_Create( p_this->p_libvlc, "httpd_mutex", VLC_VAR_MUTEX ); var_Create( p_this->p_libvlc, "httpd_mutex", VLC_VAR_MUTEX );
var_Get( p_this->p_libvlc, "httpd_mutex", &lockval ); var_Get( p_this->p_libvlc, "httpd_mutex", &lockval );
...@@ -892,6 +970,7 @@ httpd_host_t *httpd_HostNew( vlc_object_t *p_this, char *psz_host, int i_port ) ...@@ -892,6 +970,7 @@ httpd_host_t *httpd_HostNew( vlc_object_t *p_this, char *psz_host, int i_port )
if( ( httpd = vlc_object_create( p_this, VLC_OBJECT_HTTPD ) ) == NULL ) if( ( httpd = vlc_object_create( p_this, VLC_OBJECT_HTTPD ) ) == NULL )
{ {
vlc_mutex_unlock( lockval.p_address ); vlc_mutex_unlock( lockval.p_address );
freeaddrinfo( res );
return NULL; return NULL;
} }
...@@ -902,38 +981,65 @@ httpd_host_t *httpd_HostNew( vlc_object_t *p_this, char *psz_host, int i_port ) ...@@ -902,38 +981,65 @@ httpd_host_t *httpd_HostNew( vlc_object_t *p_this, char *psz_host, int i_port )
vlc_object_attach( httpd, p_this->p_vlc ); vlc_object_attach( httpd, p_this->p_vlc );
} }
/* verify if it already exist */ for( ptr = res; (ptr != NULL) && (fd == -1); ptr = ptr->ai_next )
for( i = 0; i < httpd->i_host; i++ )
{ {
vlc_bool_t b_match = VLC_FALSE; int i;
if (sock.ss_family != httpd->host[i]->sock.ss_family || if( ((unsigned)ptr->ai_addrlen) > sizeof( struct sockaddr_storage ) )
i_sock_size != httpd->host[i]->i_sock_size) {
msg_Dbg( p_this, "socket address too big" );
continue; continue;
}
switch (sock.ss_family)
/* verify if it already exist */
for( i = 0; i < httpd->i_host; i++ )
{ {
case AF_INET: if (GetAddrPort (&httpd->host[i]->sock) != i_port)
continue;
#ifdef AF_INET6
if( httpd->host[i]->sock.ss_family == AF_INET6 )
{ {
const struct sockaddr_in *p_mysock, *p_thatsock; const struct sockaddr_in6 *p_hsock, *p_sock;
p_mysock = (const struct sockaddr_in *)&sock; p_hsock = (const struct sockaddr_in6 *)&httpd->host[i]->sock;
p_thatsock = (const struct sockaddr_in *)&httpd->host[i]->sock; p_sock = (const struct sockaddr_in6 *)res->ai_addr;
b_match = p_mysock->sin_port == p_thatsock->sin_port && if( memcmp( &p_hsock->sin6_addr, &in6addr_any,
( p_mysock->sin_addr.s_addr == INADDR_ANY || sizeof( struct in6_addr ) ) &&
p_mysock->sin_addr.s_addr == ( p_sock->sin6_family != AF_INET6 ||
p_thatsock->sin_addr.s_addr ) ? memcmp( &p_hsock->sin6_addr, &p_sock->sin6_addr,
VLC_TRUE : VLC_FALSE; sizeof( struct in6_addr ) ) ) )
break; continue; /* does not match */
} }
else
default: if( res->ai_family == PF_INET6 )
msg_Dbg( p_this, "host with unknown address family" ); continue;
} else
#endif
if( httpd->host[i]->sock.ss_family == AF_INET )
{
const struct sockaddr_in *p_hsock, *p_sock;
if (b_match == VLC_TRUE) p_hsock = (const struct sockaddr_in *)&httpd->host[i]->sock;
{ p_sock = (const struct sockaddr_in *)res->ai_addr;
if( p_hsock->sin_addr.s_addr != INADDR_ANY &&
( p_sock->sin_family != AF_INET ||
p_hsock->sin_addr.s_addr != p_sock->sin_addr.s_addr ) )
continue; /* does not match */
}
else
if( res->ai_family == PF_INET )
continue;
else
{
msg_Dbg( p_this, "host with unknown address family" );
continue;
}
freeaddrinfo( res );
/* yep found */ /* yep found */
host = httpd->host[i]; host = httpd->host[i];
host->i_ref++; host->i_ref++;
...@@ -943,75 +1049,86 @@ httpd_host_t *httpd_HostNew( vlc_object_t *p_this, char *psz_host, int i_port ) ...@@ -943,75 +1049,86 @@ httpd_host_t *httpd_HostNew( vlc_object_t *p_this, char *psz_host, int i_port )
msg_Dbg( p_this, "host already registered" ); msg_Dbg( p_this, "host already registered" );
return host; return host;
} }
}
/* create the new host */
host = vlc_object_create( p_this, sizeof( httpd_host_t ) );
host->httpd = httpd;
vlc_mutex_init( httpd, &host->lock );
host->i_ref = 1;
memcpy( &host->sock, &sock, sizeof( struct sockaddr_storage ) );
host->i_sock_size = i_sock_size;
host->i_url = 0;
host->url = NULL;
host->i_client = 0;
host->client = NULL;
/* create the listening socket */ /* create the listening socket */
if( ( host->fd = socket( PF_INET, SOCK_STREAM, 0 ) ) < 0 ) fd = socket( res->ai_family, res->ai_socktype, res->ai_protocol );
{ if( fd < 0 )
goto socket_error; continue;
}
/* reuse socket */ /* reuse socket */
i = 1; {
if( setsockopt( host->fd, SOL_SOCKET, SO_REUSEADDR, int dummy = 1;
(void *) &i, sizeof( i ) ) < 0 ) if( setsockopt( fd, SOL_SOCKET, SO_REUSEADDR,
{ (void *)&dummy, sizeof( dummy ) ) < 0 )
msg_Warn( p_this, "cannot configure socket (SO_REUSEADDR)" ); {
} msg_Warn( p_this, "cannot configure socket (SO_REUSEADDR)" );
/* bind it */ }
if( bind( host->fd, (struct sockaddr *)&host->sock, host->i_sock_size ) ) }
{
msg_Err( p_this, "cannot bind socket" ); /* bind it */
goto socket_error; if( bind( fd, res->ai_addr, res->ai_addrlen ) )
} {
/* set to non-blocking */ msg_Err( p_this, "cannot bind socket" );
continue;
}
/* set to non-blocking */
#if defined( WIN32 ) || defined( UNDER_CE ) #if defined( WIN32 ) || defined( UNDER_CE )
{
unsigned long i_dummy = 1;
if( ioctlsocket( host->fd, FIONBIO, &i_dummy ) != 0 )
{ {
msg_Err( p_this, "cannot set socket to non-blocking mode" ); unsigned long i_dummy = 1;
goto socket_error; if( ioctlsocket( fd, FIONBIO, &i_dummy ) != 0 )
{
msg_Err( p_this, "cannot set socket to non-blocking mode" );
continue;
}
} }
}
#else #else
{
unsigned int i_flags;
if( ( i_flags = fcntl( host->fd, F_GETFL, 0 ) ) < 0 )
{ {
msg_Err( p_this, "cannot F_GETFL socket" ); unsigned int i_flags;
goto socket_error; if( ( i_flags = fcntl( fd, F_GETFL, 0 ) ) < 0 )
{
msg_Err( p_this, "cannot F_GETFL socket" );
continue;
}
if( fcntl( fd, F_SETFL, i_flags | O_NONBLOCK ) < 0 )
{
msg_Err( p_this, "cannot F_SETFL O_NONBLOCK" );
continue;
}
} }
if( fcntl( host->fd, F_SETFL, i_flags | O_NONBLOCK ) < 0 ) #endif
/* listen */
if( listen( fd, LISTEN_BACKLOG ) < 0 )
{ {
msg_Err( p_this, "cannot F_SETFL O_NONBLOCK" ); msg_Err( p_this, "cannot listen socket" );
goto socket_error; continue;
} }
} }
#endif
/* listen */
if( listen( host->fd, LISTEN_BACKLOG ) < 0 )
{
msg_Err( p_this, "cannot listen socket" );
goto socket_error;
}
freeaddrinfo( res );
if( fd == -1 )
goto error;
/* create the new host */
host = vlc_object_create( p_this, sizeof( httpd_host_t ) );
host->httpd = httpd;
vlc_mutex_init( httpd, &host->lock );
host->i_ref = 1;
host->fd = fd;
memcpy( &host->sock, res->ai_addr, res->ai_addrlen );
host->i_sock_size = res->ai_addrlen;
host->i_url = 0;
host->url = NULL;
host->i_client = 0;
host->client = NULL;
/* create the thread */ /* create the thread */
if( vlc_thread_create( host, "httpd host thread", httpd_HostThread, if( vlc_thread_create( host, "httpd host thread", httpd_HostThread,
VLC_THREAD_PRIORITY_LOW, VLC_FALSE ) ) VLC_THREAD_PRIORITY_LOW, VLC_FALSE ) )
{ {
msg_Err( p_this, "cannot spawn http host thread" ); msg_Err( p_this, "cannot spawn http host thread" );
goto socket_error; goto error;
} }
/* now add it to httpd */ /* now add it to httpd */
...@@ -1020,15 +1137,17 @@ httpd_host_t *httpd_HostNew( vlc_object_t *p_this, char *psz_host, int i_port ) ...@@ -1020,15 +1137,17 @@ httpd_host_t *httpd_HostNew( vlc_object_t *p_this, char *psz_host, int i_port )
return host; return host;
socket_error: error:
vlc_mutex_unlock( lockval.p_address ); vlc_mutex_unlock( lockval.p_address );
if( host->fd > 0 ) if( fd != -1 )
net_Close( fd );
if( host != NULL )
{ {
SOCKET_CLOSE( host->fd ); vlc_mutex_destroy( &host->lock );
vlc_object_destroy( host );
} }
vlc_mutex_destroy( &host->lock );
vlc_object_destroy( host );
/* TODO destroy no more used httpd TODO */ /* TODO destroy no more used httpd TODO */
vlc_object_release( httpd ); vlc_object_release( httpd );
...@@ -1081,7 +1200,7 @@ void httpd_HostDelete( httpd_host_t *host ) ...@@ -1081,7 +1200,7 @@ void httpd_HostDelete( httpd_host_t *host )
/* TODO */ /* TODO */
} }
SOCKET_CLOSE( host->fd ); net_Close( host->fd );
vlc_mutex_destroy( &host->lock ); vlc_mutex_destroy( &host->lock );
vlc_object_destroy( host ); vlc_object_destroy( host );
...@@ -1320,15 +1439,59 @@ void httpd_ClientModeBidir( httpd_client_t *cl ) ...@@ -1320,15 +1439,59 @@ void httpd_ClientModeBidir( httpd_client_t *cl )
char* httpd_ClientIP( httpd_client_t *cl ) char* httpd_ClientIP( httpd_client_t *cl )
{ {
/* FIXME not thread safe - should use inet_ntop if available */ #ifdef HAVE_GETNAMEINFO
char sz_ip[INET6_ADDRSTRLEN + 2];
int i;
if( (cl->sock.ss_family == AF_INET6) &&
IN6_IS_ADDR_V4MAPPED( &((const struct sockaddr_in6 *)
&cl->sock)->sin6_addr) )
{
/* If client is using IPv4 but server is using IPv6 */
struct sockaddr_in a;
memset( &a, 0, sizeof( a ) );
a.sin_family = AF_INET;
a.sin_port = ((const struct sockaddr_in6 *)&cl->sock)->sin6_port;
a.sin_addr.s_addr = ((const uint32_t *)&((const struct sockaddr_in6 *)
&cl->sock)->sin6_addr)[3];
i = getnameinfo( (const struct sockaddr *)&a, sizeof( a ),
&sz_ip[1], INET6_ADDRSTRLEN, NULL, 0, NI_NUMERICHOST );
}
else
i = getnameinfo( (const struct sockaddr *)&cl->sock, cl->i_sock_size,
&sz_ip[1], INET6_ADDRSTRLEN, NULL, 0,
NI_NUMERICHOST );
if( i != 0 )
/* FIXME: msg_Err */
return NULL;
fprintf( stderr, "ClientIP = %s\n", &sz_ip[1]);
if( strchr( &sz_ip[1], ':' ) != NULL )
{
*sz_ip = '[';
i = strlen( sz_ip );
sz_ip[i++] = ']';
sz_ip[i] = '\0';
fprintf( stderr, "ClientIP (with []) = %s\n", sz_ip);
return strdup( sz_ip );
}
return strdup( &sz_ip[1] );
#else
/* FIXME not thread safe */
return strdup( inet_ntoa( ((const struct sockaddr_in *)&cl->sock)->sin_addr ) ); return strdup( inet_ntoa( ((const struct sockaddr_in *)&cl->sock)->sin_addr ) );
#endif
} }
static void httpd_ClientClean( httpd_client_t *cl ) static void httpd_ClientClean( httpd_client_t *cl )
{ {
if( cl->fd > 0 ) if( cl->fd > 0 )
{ {
SOCKET_CLOSE( cl->fd ); net_Close( cl->fd );
cl->fd = -1; cl->fd = -1;
} }
...@@ -1826,8 +1989,14 @@ static void httpd_HostThread( httpd_host_t *host ) ...@@ -1826,8 +1989,14 @@ static void httpd_HostThread( httpd_host_t *host )
( cl->i_state == HTTPD_CLIENT_DEAD || ( cl->i_state == HTTPD_CLIENT_DEAD ||
cl->i_activity_date + cl->i_activity_timeout < mdate() ) ) ) cl->i_activity_date + cl->i_activity_timeout < mdate() ) ) )
{ {
char *ip;
// FIXME: it sucks to allocate memory on the stack for debug
ip = httpd_ClientIP( cl );
msg_Dbg( host, "connection closed(%s)", msg_Dbg( host, "connection closed(%s)",
inet_ntoa(((const struct sockaddr_in *)&cl->sock)->sin_addr) ); (ip != NULL) ? ip : "unknown" );
free( ip );
httpd_ClientClean( cl ); httpd_ClientClean( cl );
TAB_REMOVE( host->i_client, host->client, cl ); TAB_REMOVE( host->i_client, host->client, cl );
free( cl ); free( cl );
...@@ -2179,14 +2348,19 @@ static void httpd_HostThread( httpd_host_t *host ) ...@@ -2179,14 +2348,19 @@ static void httpd_HostThread( httpd_host_t *host )
fd = accept( host->fd, (struct sockaddr *)&sock, &i_sock_size ); fd = accept( host->fd, (struct sockaddr *)&sock, &i_sock_size );
if( fd > 0 ) if( fd > 0 )
{ {
char *ip;
httpd_client_t *cl = httpd_ClientNew( fd, &sock, i_sock_size ); httpd_client_t *cl = httpd_ClientNew( fd, &sock, i_sock_size );
vlc_mutex_lock( &host->lock ); vlc_mutex_lock( &host->lock );
TAB_APPEND( host->i_client, host->client, cl ); TAB_APPEND( host->i_client, host->client, cl );
vlc_mutex_unlock( &host->lock ); vlc_mutex_unlock( &host->lock );
// FIXME: it sucks to allocate heap memory for a debug message
ip = httpd_ClientIP( cl );
msg_Dbg( host, "new connection (%s)", msg_Dbg( host, "new connection (%s)",
inet_ntoa(((const struct sockaddr_in *)&sock)->sin_addr) ); ip != NULL ? ip : "unknown" );
free( ip );
} }
} }
/* now try all others socket */ /* now try all others socket */
...@@ -2214,17 +2388,17 @@ static void httpd_HostThread( httpd_host_t *host ) ...@@ -2214,17 +2388,17 @@ static void httpd_HostThread( httpd_host_t *host )
} }
} }
static int BuildAddr( struct sockaddr_storage * p_socket, int *pi_sock_size, #ifndef HAVE_GETADDRINFO
static int BuildAddr( struct sockaddr_in * p_socket,
const char * psz_address, int i_port ) const char * psz_address, int i_port )
{ {
/* Reset struct */ /* Reset struct */
memset( p_socket, 0, sizeof( struct sockaddr_storage ) ); memset( p_socket, 0, sizeof( struct sockaddr_in ) );
p_socket->ss_family = AF_INET; /* family */ p_socket->sin_family = AF_INET; /* family */
*pi_sock_size = sizeof( struct sockaddr_in ); p_socket->sin_port = htons( (uint16_t)i_port );
((struct sockaddr_in *)p_socket)->sin_port = htons( (uint16_t)i_port );
if( !*psz_address ) if( !*psz_address )
{ {
((struct sockaddr_in *)p_socket)->sin_addr.s_addr = INADDR_ANY; p_socket->sin_addr.s_addr = INADDR_ANY;
} }
else else
{ {
...@@ -2233,11 +2407,12 @@ static int BuildAddr( struct sockaddr_storage * p_socket, int *pi_sock_size, ...@@ -2233,11 +2407,12 @@ static int BuildAddr( struct sockaddr_storage * p_socket, int *pi_sock_size,
/* Try to convert address directly from in_addr - this will work if /* Try to convert address directly from in_addr - this will work if
* psz_address is dotted decimal. */ * psz_address is dotted decimal. */
#ifdef HAVE_ARPA_INET_H #ifdef HAVE_ARPA_INET_H
if( !inet_aton( psz_address, &((struct sockaddr_in *)p_socket)->sin_addr ) ) if( !inet_aton( psz_address, &p_socket->sin_addr ) )
#else #else
((struct sockaddr_in *)p_socket)->sin_addr.s_addr = inet_addr( psz_address ); p_socket->sin_addr.s_addr = inet_addr( psz_address );
/* if( ((struct sockaddr_in *)p_socket)->sin_addr.s_addr == INADDR_NONE )*/
if( ((struct sockaddr_in *)p_socket)->sin_addr.s_addr == INADDR_BROADCAST ) /* if( p_socket->sin_addr.s_addr == INADDR_NONE )*/
if( p_socket->sin_addr.s_addr == INADDR_BROADCAST )
#endif #endif
{ {
/* We have a fqdn, try to find its address */ /* We have a fqdn, try to find its address */
...@@ -2253,3 +2428,27 @@ static int BuildAddr( struct sockaddr_storage * p_socket, int *pi_sock_size, ...@@ -2253,3 +2428,27 @@ static int BuildAddr( struct sockaddr_storage * p_socket, int *pi_sock_size,
} }
return( 0 ); return( 0 );
} }
#endif
static int GetAddrPort( const struct sockaddr_storage *p_ss )
{
int i_port = 0;
switch (p_ss->ss_family)
{
#ifdef AF_INET6
case AF_INET6:
i_port = ((const struct sockaddr_in6 *)p_ss)->sin6_port;
break;
#endif
case AF_INET:
i_port = ((const struct sockaddr_in *)p_ss)->sin_port;
break;
default:
return -1;
}
return ntohs( i_port );
}
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