I've managed to get three -1s off this (Jeff, Ryan, and myself).

Revert the context null check.  My bad.


git-svn-id: https://svn.apache.org/repos/asf/apr/apr/trunk@61694 13f79535-47bb-0310-9956-ffa450edef68

1 files changed, 0 insertions, 12 deletions

diff --git a/passwd/apr_md5.c b/passwd/apr_md5.c
index 55251c368..9ce2e3f05 100644
--- a/passwd/apr_md5.c
+++ b/passwd/apr_md5.c
@@ -188,9 +188,6 @@ static apr_xlate_t *xlate_ebcdic_to_ascii; /* used in apr_md5_encode() */
  */
 APR_DECLARE(apr_status_t) apr_md5_init(apr_md5_ctx_t *context)
 {
-    if (!context)
-        return APR_EINVAL;
-    
     context->count[0] = context->count[1] = 0;
     
     /* Load magic initialization constants. */
@@ -217,9 +214,6 @@ APR_DECLARE(apr_status_t) apr_md5_set_xlate(apr_md5_ctx_t *context,
     apr_status_t rv;
     int is_sb;
 
-    if (!context)
-        return APR_EINVAL;
-    
     /* TODO: remove the single-byte-only restriction from this code
      */
     rv = apr_xlate_get_sb(xlate, &is_sb);
@@ -247,9 +241,6 @@ APR_DECLARE(apr_status_t) apr_md5_update(apr_md5_ctx_t *context,
     apr_size_t inbytes_left, outbytes_left;
 #endif
 
-    if (!context)
-        return APR_EINVAL;
-    
     /* Compute number of bytes mod 64 */
     idx = (unsigned int)((context->count[0] >> 3) & 0x3F);
 
@@ -329,9 +320,6 @@ APR_DECLARE(apr_status_t) apr_md5_final(unsigned char digest[MD5_DIGESTSIZE],
     unsigned char bits[8];
     unsigned int idx, padLen;
 
-    if (!context)
-        return APR_EINVAL;
-    
     /* Save number of bits */
     Encode(bits, context->count, 8);