From ebe258dd4091c00cf2ca3d6648cf4d60cb682ee1 Mon Sep 17 00:00:00 2001 From: Stefan Metzmacher Date: Thu, 21 Jul 2011 21:47:36 +0200 Subject: s3:libsmb: replace cli_initialise[_ex]() by cli_state_create() This makes sure cli_state->src_ss and cli_state->dest_ss are always initialized. metze --- source3/libsmb/cliconnect.c | 26 ++------------------------ 1 file changed, 2 insertions(+), 24 deletions(-) (limited to 'source3/libsmb/cliconnect.c') diff --git a/source3/libsmb/cliconnect.c b/source3/libsmb/cliconnect.c index 5317c192bc..6a7931b786 100644 --- a/source3/libsmb/cliconnect.c +++ b/source3/libsmb/cliconnect.c @@ -2849,8 +2849,6 @@ NTSTATUS cli_connect_nb(const char *host, struct sockaddr_storage *pss, int fd = -1; char *desthost; char *p; - socklen_t length; - int ret; desthost = talloc_strdup(talloc_tos(), host); if (desthost == NULL) { @@ -2866,34 +2864,14 @@ NTSTATUS cli_connect_nb(const char *host, struct sockaddr_storage *pss, } } - cli = cli_initialise_ex(signing_state); - if (cli == NULL) { - goto fail; - } - cli->desthost = talloc_move(cli, &desthost); - status = cli_connect_sock(host, name_type, pss, myname, port, 20, &fd, &port); if (!NT_STATUS_IS_OK(status)) { - cli_shutdown(cli); goto fail; } - cli->fd = fd; - length = sizeof(cli->src_ss); - ret = getsockname(fd, (struct sockaddr *)(void *)&cli->src_ss, - &length); - if (ret == -1) { - status = map_nt_error_from_unix(errno); - cli_shutdown(cli); - goto fail; - } - length = sizeof(cli->dest_ss); - ret = getpeername(fd, (struct sockaddr *)(void *)&cli->dest_ss, - &length); - if (ret == -1) { - status = map_nt_error_from_unix(errno); - cli_shutdown(cli); + cli = cli_state_create(NULL, fd, desthost, signing_state); + if (cli == NULL) { goto fail; } -- cgit