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/modules/ssl@97372 13f79535-47bb-0310-9956-ffa450edef68

diff --git a/ssl_engine_io.c b/ssl_engine_io.c
index 3928b892dcd1d29c973b32ee134ae900e38e68a8..7718671427a48160bc9693046da7a093ac033c62 100644 (file)
--- a/ssl_engine_io.c
+++ b/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;
         }