diff options
| author | Matt Caswell <matt@openssl.org> | 2015-03-24 15:10:15 +0000 |
|---|---|---|
| committer | Matt Caswell <matt@openssl.org> | 2015-03-25 18:52:13 +0000 |
| commit | 17dd65e6e1f888b4561d559b4d44fbbe0a0aa3e7 (patch) | |
| tree | e1b6a255d4d7dd7eb37734b4208af8598a3078be /ssl/d1_pkt.c | |
| parent | cb14aec713ccab92d1c21d3d5b066570c4bca5ee (diff) | |
Resolve swallowed returns codes
The recent updates to libssl to enforce stricter return code checking, left
a small number of instances behind where return codes were being swallowed
(typically because the function they were being called from was declared as
void). This commit fixes those instances to handle the return codes more
appropriately.
Reviewed-by: Richard Levitte <levitte@openssl.org>
Diffstat (limited to 'ssl/d1_pkt.c')
| -rw-r--r-- | ssl/d1_pkt.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/ssl/d1_pkt.c b/ssl/d1_pkt.c index 5463acfe7f..19e60b7889 100644 --- a/ssl/d1_pkt.c +++ b/ssl/d1_pkt.c @@ -1249,8 +1249,7 @@ int dtls1_read_bytes(SSL *s, int type, unsigned char *buf, int len, int peek) if (dtls1_check_timeout_num(s) < 0) return -1; - /* Ignore retransmit failures - swallow return code */ - if(dtls1_retransmit_buffered_messages(s)); + dtls1_retransmit_buffered_messages(s); rr->length = 0; goto start; } |