Skip to content

Commit

Permalink
* ext/openssl/ossl_ssl.c: Revert r35583
Browse files Browse the repository at this point in the history
* test/openssl/test_ssl.rb: Handle ECONNRESET in code instead to avoid
the test failing in Ruby CI [1]
	
[1] http://u64.rubyci.org/~chkbuild/ruby-trunk/log/20120507T190102Z.log.html.gz#test-all


git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@35795 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
  • Loading branch information
emboss committed May 25, 2012
1 parent 655a20e commit 913827b
Show file tree
Hide file tree
Showing 3 changed files with 11 additions and 0 deletions.
8 changes: 8 additions & 0 deletions ChangeLog
Original file line number Diff line number Diff line change
@@ -1,3 +1,11 @@
Fri May 25 22:19:40 2012 Martin Bosslet <Martin.Bosslet@googlemail.com>

* ext/openssl/ossl_ssl.c: Revert r35583
* test/openssl/test_ssl.rb: Handle ECONNRESET in code instead to avoid
the test failing in Ruby CI [1]

[1] http://u64.rubyci.org/~chkbuild/ruby-trunk/log/20120507T190102Z.log.html.gz#test-all

Fri May 25 19:51:36 2012 Koichi Sasada <ko1@atdot.net>

* vm_eval.c (rb_f_caller): caller() method accepts second optional
Expand Down
1 change: 1 addition & 0 deletions ext/openssl/ossl_ssl.c
Original file line number Diff line number Diff line change
Expand Up @@ -1158,6 +1158,7 @@ ossl_start_ssl(VALUE self, int (*func)(), const char *funcname, int nonblock)
rb_io_wait_readable(FPTR_TO_FD(fptr));
continue;
case SSL_ERROR_SYSCALL:
if (errno) rb_sys_fail(funcname);
ossl_raise(eSSLError, "%s SYSCALL returned=%d errno=%d state=%s", funcname, ret2, errno, SSL_state_string_long(ssl));
default:
ossl_raise(eSSLError, "%s returned=%d errno=%d state=%s", funcname, ret2, errno, SSL_state_string_long(ssl));
Expand Down
2 changes: 2 additions & 0 deletions test/openssl/test_ssl.rb
Original file line number Diff line number Diff line change
Expand Up @@ -516,6 +516,8 @@ def server_connect(port, ctx=nil)
ssl.sync_close = true
ssl.connect
yield ssl
rescue Errno::ECONNRESET => e
raise OpenSSL::SSL::SSLError.new(e.message)
ensure
ssl.close
end
Expand Down

0 comments on commit 913827b

Please sign in to comment.