diff options
| author | Matt Caswell <matt@openssl.org> | 2016-03-07 10:06:42 +0000 |
|---|---|---|
| committer | Matt Caswell <matt@openssl.org> | 2016-03-07 21:39:28 +0000 |
| commit | e2d5183d7cd5479d7c63ba524b3ddc4abd707dba (patch) | |
| tree | dd546fe84fb85c5a09963d00655839dc9734d1a3 | |
| parent | ccd82ef4c2ca0b1e931e8f6e5f3848a2110b8680 (diff) | |
Fix s_server/s_client handling of the split_send_frag argument
Ensure that a value of 0 is correctly handled for the split_send_frag
argument.
Reviewed-by: Tim Hudson <tjh@openssl.org>
| -rw-r--r-- | apps/s_client.c | 7 | ||||
| -rw-r--r-- | apps/s_server.c | 7 |
2 files changed, 10 insertions, 4 deletions
diff --git a/apps/s_client.c b/apps/s_client.c index 5a5a90b1cb..a1ef64b13f 100644 --- a/apps/s_client.c +++ b/apps/s_client.c @@ -1389,8 +1389,11 @@ int s_client_main(int argc, char **argv) case OPT_SPLIT_SEND_FRAG: split_send_fragment = atoi(opt_arg()); if (split_send_fragment == 0) { - /* Not allowed - set to a deliberately bad value */ - split_send_fragment = -1; + /* + * Not allowed - set to a deliberately bad value so we get an + * error message below + */ + split_send_fragment = SSL3_RT_MAX_PLAIN_LENGTH + 1; } break; case OPT_MAX_PIPELINES: diff --git a/apps/s_server.c b/apps/s_server.c index 870c7b24f9..35a22f7900 100644 --- a/apps/s_server.c +++ b/apps/s_server.c @@ -1517,8 +1517,11 @@ int s_server_main(int argc, char *argv[]) case OPT_SPLIT_SEND_FRAG: split_send_fragment = atoi(opt_arg()); if (split_send_fragment == 0) { - /* Not allowed - set to a deliberately bad value */ - split_send_fragment = -1; + /* + * Not allowed - set to a deliberately bad value so we get an + * error message below + */ + split_send_fragment = SSL3_RT_MAX_PLAIN_LENGTH + 1; } break; case OPT_MAX_PIPELINES: |