From b181a0b96b8313206923e6f5b46b0803aaf64907 Mon Sep 17 00:00:00 2001 From: Andrew Bartlett Date: Fri, 27 Jul 2012 12:31:28 +1000 Subject: lib/param: Remove use of lp{cfg,}_socket_address outside the NBT client and server In these other cases, control of the sockets to bind to can be obtained using "bind interfaces only = yes" and "interfaces = ". Andrew Bartlett --- source3/rpc_server/rpc_sock_helper.c | 18 ++++++------------ source3/smbd/server.c | 9 +++------ source4/lib/socket/interface.c | 10 ---------- source4/wrepl_server/wrepl_in_connection.c | 5 +---- 4 files changed, 10 insertions(+), 32 deletions(-) diff --git a/source3/rpc_server/rpc_sock_helper.c b/source3/rpc_server/rpc_sock_helper.c index c767cd817d..198df903ad 100644 --- a/source3/rpc_server/rpc_sock_helper.c +++ b/source3/rpc_server/rpc_sock_helper.c @@ -98,18 +98,15 @@ NTSTATUS rpc_create_tcpip_sockets(const struct ndr_interface_table *iface, } } } else { - const char *sock_addr = lp_socket_address(); + const char *sock_addr; const char *sock_ptr; char *sock_tok; - if (strequal(sock_addr, "0.0.0.0") || - strequal(sock_addr, "::")) { #if HAVE_IPV6 - sock_addr = "::,0.0.0.0"; + sock_addr = "::,0.0.0.0"; #else - sock_addr = "0.0.0.0"; + sock_addr = "0.0.0.0"; #endif - } for (sock_ptr = sock_addr; next_token_talloc(talloc_tos(), &sock_ptr, &sock_tok, " \t,"); @@ -217,18 +214,15 @@ NTSTATUS rpc_setup_tcpip_sockets(struct tevent_context *ev_ctx, } } } else { - const char *sock_addr = lp_socket_address(); + const char *sock_addr; const char *sock_ptr; char *sock_tok; - if (strequal(sock_addr, "0.0.0.0") || - strequal(sock_addr, "::")) { #if HAVE_IPV6 - sock_addr = "::,0.0.0.0"; + sock_addr = "::,0.0.0.0"; #else - sock_addr = "0.0.0.0"; + sock_addr = "0.0.0.0"; #endif - } for (sock_ptr = sock_addr; next_token_talloc(talloc_tos(), &sock_ptr, &sock_tok, " \t,"); diff --git a/source3/smbd/server.c b/source3/smbd/server.c index 63edf0020a..6abf8ccaeb 100644 --- a/source3/smbd/server.c +++ b/source3/smbd/server.c @@ -796,18 +796,15 @@ static bool open_sockets_smbd(struct smbd_parent_context *parent, /* Just bind to 0.0.0.0 - accept connections from anywhere. */ - const char *sock_addr = lp_socket_address(); + const char *sock_addr; char *sock_tok; const char *sock_ptr; - if (strequal(sock_addr, "0.0.0.0") || - strequal(sock_addr, "::")) { #if HAVE_IPV6 - sock_addr = "::,0.0.0.0"; + sock_addr = "::,0.0.0.0"; #else - sock_addr = "0.0.0.0"; + sock_addr = "0.0.0.0"; #endif - } for (sock_ptr=sock_addr; next_token_talloc(talloc_tos(), &sock_ptr, &sock_tok, " \t,"); ) { diff --git a/source4/lib/socket/interface.c b/source4/lib/socket/interface.c index 27ac7151a3..7a4733f466 100644 --- a/source4/lib/socket/interface.c +++ b/source4/lib/socket/interface.c @@ -502,20 +502,10 @@ bool iface_list_same_net(const char *ip1, const char *ip2, const char *netmask) /** return the list of wildcard interfaces this will include the IPv4 0.0.0.0, and may include IPv6 :: - it is overridden by the 'socket address' option in smb.conf */ const char **iface_list_wildcard(TALLOC_CTX *mem_ctx, struct loadparm_context *lp_ctx) { const char **ret; - const char *socket_address; - - /* the user may have configured a specific address */ - socket_address = lpcfg_socket_address(lp_ctx); - if (strcmp(socket_address, "") != 0) { - ret = (const char **)str_list_make(mem_ctx, socket_address, NULL); - return ret; - } - ret = (const char **)str_list_make(mem_ctx, "0.0.0.0", NULL); if (ret == NULL) return NULL; diff --git a/source4/wrepl_server/wrepl_in_connection.c b/source4/wrepl_server/wrepl_in_connection.c index 962a1cb7fa..251e5a0247 100644 --- a/source4/wrepl_server/wrepl_in_connection.c +++ b/source4/wrepl_server/wrepl_in_connection.c @@ -467,10 +467,7 @@ NTSTATUS wreplsrv_setup_sockets(struct wreplsrv_service *service, struct loadpar } } } else { - address = lpcfg_socket_address(lp_ctx); - if (strcmp(address, "") == 0) { - address = "0.0.0.0"; - } + address = "0.0.0.0"; status = stream_setup_socket(task, task->event_ctx, task->lp_ctx, model_ops, &wreplsrv_stream_ops, "ipv4", address, &port, lpcfg_socket_options(task->lp_ctx), -- cgit