From: Justin Erenkrantz <jerenkrantz@apache.org>
Date: Fri, 1 Nov 2002 09:42:09 +0000 (+0000)
Subject: I don't see how *len > wanted, but leave the check there.
X-Git-Tag: 2.0.44~174
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=35442600da56db1e4ee8e26170cb3857f3469c86;p=thirdparty%2Fapache%2Fhttpd.git

I don't see how *len > wanted, but leave the check there.

Stick a comment in there as a 'Waldo was here' so that if I ever see this
again, I realize that I've actually thought about it and didn't think >
was necessary.


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

diff --git a/modules/ssl/ssl_engine_io.c b/modules/ssl/ssl_engine_io.c
index 3928b892dcd..7718671427a 100644
--- a/modules/ssl/ssl_engine_io.c
+++ b/modules/ssl/ssl_engine_io.c
@@ -655,7 +655,10 @@ static apr_status_t ssl_io_input_read(bio_filter_in_ctx_t *inctx,
             inctx->cbuf.value -= bytes;
             inctx->cbuf.length += bytes;
             return APR_SUCCESS;
-        } 
+        }
+        /* This could probably be *len == wanted, but be safe from stray
+         * photons.
+         */
         if (*len >= wanted) {
             return APR_SUCCESS;
         }