Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(870)

Issue 2341: avoid ssl_eof error on SSL_WANT_READ and friends

Can't Edit
Can't Publish+Mail
Start Review
Created:
5 years, 10 months ago by Jacob
Modified:
4 years, 8 months ago
Reviewers:
dangabrad, djabberd
Base URL:
http://code.sixapart.com/svn/djabberd/trunk/DJabberd
Visibility:
Public.

Description

Incrementing a counter on empty SSL reads is not the correct way to determine
ssl_eof
(http://code.sixapart.com/trac/djabberd/changeset/758)

Instead we should check the error code to see if it is one of the SSL errors for
which we are supposed to retry(
http://www.openssl.org/docs/ssl/SSL_get_error.html)

Patch Set 1 #

Patch Set 2 : fixes #

Patch Set 3 : response to comments #

Unified diffs Side-by-side diffs Delta from patch set Stats (+24 lines, -1 line) Patch
lib/DJabberd/Connection.pm View 1 2 1 chunk +5 lines, -1 line 0 comments Download
lib/DJabberd/Stanza/StartTLS.pm View 1 2 chunks +19 lines, -0 lines 0 comments Download

Messages

Total messages: 3
dangabrad
http://codereview.appspot.com/2341/diff/1/2 File lib/DJabberd/Stanza/StartTLS.pm (right): http://codereview.appspot.com/2341/diff/1/2#newcode74 Line 74: my $err = Net::SSLeay::get_error($ssl,-1); space after comma http://codereview.appspot.com/2341/diff/1/2#newcode85 ...
5 years, 10 months ago #1
Jacob
I've uploaded a new patch http://codereview.appspot.com/2341/diff/1/2 File lib/DJabberd/Stanza/StartTLS.pm (right): http://codereview.appspot.com/2341/diff/1/2#newcode74 Line 74: my $err = ...
5 years, 9 months ago #2
dangabrad
5 years, 9 months ago #3
http://codereview.appspot.com/2341/diff/61/82
File lib/DJabberd/Connection.pm (right):

http://codereview.appspot.com/2341/diff/61/82#newcode495
Line 495: # A call to 'should_we_close_on_zero_bytes' accounts for the
possibility
You didn't update this comment.

Otherwise this seems fine now, assuming you've tested it all.  See
http://danga.com/commit.bml and let's get you commit access.
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld 1274:9b9295c7fd64