]>
Commit | Line | Data |
---|---|---|
13ce82d8 | 1 | From: msvensson at mysql dot com |
2 | Date: December 22 2006 12:05am | |
3 | Subject: bk commit into 5.0 tree (msvensson:1.2352) BUG#24148 | |
4 | ||
5 | Below is the list of changes that have just been committed into a local | |
6 | 5.0 repository of msvensson. When msvensson does a push these changes will | |
7 | be propagated to the main repository and, within 24 hours after the | |
8 | push, to the public repository. | |
9 | For information on how to access the public repository | |
10 | see http://dev.mysql.com/doc/mysql/en/installing-source-tree.html | |
11 | ||
12 | ChangeSet@stripped, 2006-12-22 00:05:40+01:00, msvensson@stripped +1 -0 | |
13 | Bug#24148 regression tests hang with SSL enabled | |
14 | - Don't call SSL_shutdown a second time | |
15 | ||
16 | vio/viossl.c@stripped, 2006-12-22 00:05:39+01:00, msvensson@stripped +9 -5 | |
17 | SSL_shutdown might return 0 indicating that it should be called once | |
18 | again for a graceful shutdown. Since the socket are going to be closed | |
19 | anyway ther is no need for the second call. | |
20 | ||
21 | # This is a BitKeeper patch. What follows are the unified diffs for the | |
22 | # set of deltas contained in the patch. The rest of the patch, the part | |
23 | # that BitKeeper cares about, is below these diffs. | |
24 | # User: msvensson | |
25 | # Host: pilot.mysql.com | |
26 | # Root: /home/msvensson/mysql/mysql-5.0-maint | |
27 | ||
28 | --- 1.38/vio/viossl.c 2006-11-20 21:41:56 +01:00 | |
29 | +++ 1.39/vio/viossl.c 2006-12-22 00:05:39 +01:00 | |
30 | @@ -126,12 +126,16 @@ | |
31 | { | |
32 | switch ((r= SSL_shutdown(ssl))) | |
33 | { | |
34 | - case 1: /* Shutdown successful */ | |
35 | + case 1: | |
36 | + /* Shutdown successful */ | |
37 | + break; | |
38 | + case 0: | |
39 | + /* | |
40 | + Shutdown not yet finished - since the socket is going to | |
41 | + be closed there is no need to call SSL_shutdown() a second | |
42 | + time to wait for the other side to respond | |
43 | + */ | |
44 | break; | |
45 | - case 0: /* Shutdown not yet finished, call it again */ | |
46 | - if ((r= SSL_shutdown(ssl) >= 0)) | |
47 | - break; | |
48 | - /* Fallthrough */ | |
49 | default: /* Shutdown failed */ | |
50 | DBUG_PRINT("vio_error", ("SSL_shutdown() failed, error: %d", | |
51 | SSL_get_error(ssl, r))); |