# HG changeset patch # User Sergey Kandaurov # Date 1606178636 0 # Node ID fd440d32470046bc2ab6e214198fbae4f3d3c937 # Parent 166461f0bd4b0934d1531e54214be56a0365d32d Tests: simplified get_ssl_socket() functions that use Net::SSLeay. diff --git a/mail_ssl.t b/mail_ssl.t --- a/mail_ssl.t +++ b/mail_ssl.t @@ -12,8 +12,6 @@ use strict; use Test::More; -use Socket qw/ :DEFAULT $CRLF /; - BEGIN { use FindBin; chdir($FindBin::Bin); } use lib 'lib'; @@ -294,14 +292,8 @@ like(Net::SSLeay::dump_peer_certificate( sub get_ssl_socket { my ($port, $ses) = @_; - my $s; - my $dest_ip = inet_aton('127.0.0.1'); - my $dest_serv_params = sockaddr_in(port($port), $dest_ip); - - socket($s, &AF_INET, &SOCK_STREAM, 0) or die "socket: $!"; - connect($s, $dest_serv_params) or die "connect: $!"; - + my $s = IO::Socket::INET->new('127.0.0.1:' . port($port)); my $ssl = Net::SSLeay::new($ctx) or die("Failed to create SSL $!"); Net::SSLeay::set_session($ssl, $ses) if defined $ses; Net::SSLeay::set_fd($ssl, fileno($s)); diff --git a/ssl_certificate.t b/ssl_certificate.t --- a/ssl_certificate.t +++ b/ssl_certificate.t @@ -12,7 +12,7 @@ use strict; use Test::More; -use Socket qw/ :DEFAULT CRLF /; +use Socket qw/ CRLF /; BEGIN { use FindBin; chdir($FindBin::Bin); } @@ -214,15 +214,8 @@ sub cert { sub get_ssl_socket { my ($host, $port, $ses) = @_; - my $s; - my $dest_ip = inet_aton('127.0.0.1'); - $port = port($port); - my $dest_serv_params = sockaddr_in($port, $dest_ip); - - socket($s, &AF_INET, &SOCK_STREAM, 0) or die "socket: $!"; - connect($s, $dest_serv_params) or die "connect: $!"; - + my $s = IO::Socket::INET->new('127.0.0.1:' . port($port)); my $ctx = Net::SSLeay::CTX_new() or die("Failed to create SSL_CTX $!"); my $ssl = Net::SSLeay::new($ctx) or die("Failed to create SSL $!"); Net::SSLeay::set_tlsext_host_name($ssl, $host); diff --git a/ssl_certificate_perl.t b/ssl_certificate_perl.t --- a/ssl_certificate_perl.t +++ b/ssl_certificate_perl.t @@ -12,8 +12,6 @@ use strict; use Test::More; -use Socket; - BEGIN { use FindBin; chdir($FindBin::Bin); } use lib 'lib'; @@ -114,15 +112,8 @@ sub cert { sub get_ssl_socket { my ($host, $port) = @_; - my $s; - my $dest_ip = inet_aton('127.0.0.1'); - $port = port($port); - my $dest_serv_params = sockaddr_in($port, $dest_ip); - - socket($s, &AF_INET, &SOCK_STREAM, 0) or die "socket: $!"; - connect($s, $dest_serv_params) or die "connect: $!"; - + my $s = IO::Socket::INET->new('127.0.0.1:' . port($port)); my $ctx = Net::SSLeay::CTX_new() or die("Failed to create SSL_CTX $!"); my $ssl = Net::SSLeay::new($ctx) or die("Failed to create SSL $!"); Net::SSLeay::set_tlsext_host_name($ssl, $host); diff --git a/ssl_sni_reneg.t b/ssl_sni_reneg.t --- a/ssl_sni_reneg.t +++ b/ssl_sni_reneg.t @@ -12,7 +12,7 @@ use strict; use Test::More; -use Socket qw/ :DEFAULT CRLF /; +use Socket qw/ CRLF /; BEGIN { use FindBin; chdir($FindBin::Bin); } @@ -150,15 +150,11 @@ sub get_ssl_socket { my ($port) = @_; my $s; - my $dest_ip = inet_aton('127.0.0.1'); - my $dest_serv_params = sockaddr_in(port($port), $dest_ip); - eval { local $SIG{ALRM} = sub { die "timeout\n" }; local $SIG{PIPE} = sub { die "sigpipe\n" }; alarm(8); - socket($s, &AF_INET, &SOCK_STREAM, 0) or die "socket: $!"; - connect($s, $dest_serv_params) or die "connect: $!"; + $s = IO::Socket::INET->new('127.0.0.1:' . port($port)); alarm(0); }; alarm(0); diff --git a/ssl_verify_client.t b/ssl_verify_client.t --- a/ssl_verify_client.t +++ b/ssl_verify_client.t @@ -12,7 +12,7 @@ use strict; use Test::More; -use Socket qw/ :DEFAULT CRLF /; +use Socket qw/ CRLF /; BEGIN { use FindBin; chdir($FindBin::Bin); } @@ -185,12 +185,7 @@ sub get { $host = $sni if !defined $host; - my $dest_ip = inet_aton('127.0.0.1'); - my $dest_serv_params = sockaddr_in(port(8081), $dest_ip); - - socket(my $s, &AF_INET, &SOCK_STREAM, 0) or die "socket: $!"; - connect($s, $dest_serv_params) or die "connect: $!"; - + my $s = IO::Socket::INET->new('127.0.0.1:' . port(8081)); my $ctx = Net::SSLeay::CTX_new() or die("Failed to create SSL_CTX $!"); Net::SSLeay::set_cert_and_key($ctx, "$d/$cert.crt", "$d/$cert.key") or die if $cert; diff --git a/stream_ssl.t b/stream_ssl.t --- a/stream_ssl.t +++ b/stream_ssl.t @@ -13,7 +13,7 @@ use strict; use Test::More; use POSIX qw/ mkfifo /; -use Socket qw/ :DEFAULT $CRLF /; +use Socket qw/ $CRLF /; BEGIN { use FindBin; chdir($FindBin::Bin); } @@ -182,14 +182,8 @@ like(Net::SSLeay::dump_peer_certificate( sub get_ssl_socket { my ($port, $ses) = @_; - my $s; - my $dest_ip = inet_aton('127.0.0.1'); - my $dest_serv_params = sockaddr_in($port, $dest_ip); - - socket($s, &AF_INET, &SOCK_STREAM, 0) or die "socket: $!"; - connect($s, $dest_serv_params) or die "connect: $!"; - + my $s = IO::Socket::INET->new('127.0.0.1:' . $port); my $ssl = Net::SSLeay::new($ctx) or die("Failed to create SSL $!"); Net::SSLeay::set_session($ssl, $ses) if defined $ses; Net::SSLeay::set_fd($ssl, fileno($s)); diff --git a/stream_ssl_certificate.t b/stream_ssl_certificate.t --- a/stream_ssl_certificate.t +++ b/stream_ssl_certificate.t @@ -12,8 +12,6 @@ use strict; use Test::More; -use Socket qw/ :DEFAULT CRLF /; - BEGIN { use FindBin; chdir($FindBin::Bin); } use lib 'lib'; @@ -192,15 +190,8 @@ sub cert { sub get_ssl_socket { my ($host, $port, $ses) = @_; - my $s; - my $dest_ip = inet_aton('127.0.0.1'); - $port = port($port); - my $dest_serv_params = sockaddr_in($port, $dest_ip); - - socket($s, &AF_INET, &SOCK_STREAM, 0) or die "socket: $!"; - connect($s, $dest_serv_params) or die "connect: $!"; - + my $s = IO::Socket::INET->new('127.0.0.1:' . port($port)); my $ctx = Net::SSLeay::CTX_new() or die("Failed to create SSL_CTX $!"); my $ssl = Net::SSLeay::new($ctx) or die("Failed to create SSL $!"); Net::SSLeay::set_tlsext_host_name($ssl, $host); diff --git a/stream_ssl_variables.t b/stream_ssl_variables.t --- a/stream_ssl_variables.t +++ b/stream_ssl_variables.t @@ -12,8 +12,6 @@ use strict; use Test::More; -use Socket; - BEGIN { use FindBin; chdir($FindBin::Bin); } use lib 'lib'; @@ -127,14 +125,8 @@ is(Net::SSLeay::ssl_read_all($ssl), '', sub get_ssl_socket { my ($port, $ses, $name) = @_; - my $s; - my $dest_ip = inet_aton('127.0.0.1'); - my $dest_serv_params = sockaddr_in($port, $dest_ip); - - socket($s, &AF_INET, &SOCK_STREAM, 0) or die "socket: $!"; - connect($s, $dest_serv_params) or die "connect: $!"; - + my $s = IO::Socket::INET->new('127.0.0.1:' . $port); my $ctx = Net::SSLeay::CTX_new() or die("Failed to create SSL_CTX $!"); my $ssl = Net::SSLeay::new($ctx) or die("Failed to create SSL $!"); Net::SSLeay::set_tlsext_host_name($ssl, $name) if defined $name; diff --git a/stream_ssl_verify_client.t b/stream_ssl_verify_client.t --- a/stream_ssl_verify_client.t +++ b/stream_ssl_verify_client.t @@ -13,8 +13,6 @@ use strict; use Test::More; -use Socket; - BEGIN { use FindBin; chdir($FindBin::Bin); } use lib 'lib'; @@ -142,12 +140,7 @@ is($t->read_file('status.log'), "500\n20 sub get { my ($port, $cert) = @_; - my $dest_ip = inet_aton('127.0.0.1'); - my $dest_serv_params = sockaddr_in(port($port), $dest_ip); - - socket(my $s, &AF_INET, &SOCK_STREAM, 0) or die "socket: $!"; - connect($s, $dest_serv_params) or die "connect: $!"; - + my $s = IO::Socket::INET->new('127.0.0.1:' . port($port)); my $ctx = Net::SSLeay::CTX_new() or die("Failed to create SSL_CTX $!"); Net::SSLeay::set_cert_and_key($ctx, "$d/$cert.crt", "$d/$cert.key") or die if $cert;