From: Emilia Kasper <emilia@openssl.org>
Date: Wed, 4 Mar 2015 21:05:53 +0000 (-0800)
Subject: Harmonize return values in dtls1_buffer_record
X-Git-Tag: OpenSSL_1_0_2a~42
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=0c14565c5b7683e4bd47cf299b2f6d4bc26d61aa;p=openssl

Harmonize return values in dtls1_buffer_record

Ensure all malloc failures return -1.

Reported by Adam Langley (Google).

Reviewed-by: Matt Caswell <matt@openssl.org>
(cherry picked from commit 06c6a2b4a3a6e64303caa256398dd2dc16f9c35a)
---

diff --git a/ssl/d1_pkt.c b/ssl/d1_pkt.c
index bf96f60911..940ca69279 100644
--- a/ssl/d1_pkt.c
+++ b/ssl/d1_pkt.c
@@ -236,7 +236,7 @@ dtls1_buffer_record(SSL *s, record_pqueue *queue, unsigned char *priority)
             pitem_free(item);
 
         SSLerr(SSL_F_DTLS1_BUFFER_RECORD, ERR_R_INTERNAL_ERROR);
-        return (0);
+        return -1;
     }
 
     rdata->packet = s->packet;