Commit 12f2c943 authored by Justin Erenkrantz's avatar Justin Erenkrantz
Browse files

Lose the magic number and use the central #define AP_IOBUFSIZE. 

Hey, I've never seen it in actual use anywhere, so I didn't know.

Thanks to Roy for pointing it out.


git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk/modules/ssl@91440 13f79535-47bb-0310-9956-ffa450edef68
parent c4150ee9

ssl_engine_io.c

+5 −6
Changes for ssl_engine_io.c: 5 added lines, 6 removed lines.
Original line number Diff line number Diff line
@@ -309,7 +309,7 @@ static apr_status_t churn_input(SSLFilterRec *pRec, ap_input_mode_t eMode, 
            return APR_SUCCESS;

        }

        if (rv == SSL_ERROR_WANT_READ) {

            apr_off_t tempread = 1024;

            apr_off_t tempread = AP_IOBUFSIZE;

            return churn_input(pRec, eMode, &tempread);

        }

        return rv;
@@ -331,7 +331,7 @@ static apr_status_t churn_input(SSLFilterRec *pRec, ap_input_mode_t eMode, 
    }



    if (n < 0 && errno == EINTR && APR_BRIGADE_EMPTY(ctx->b)) {

        apr_off_t tempread = 1024;

        apr_off_t tempread = AP_IOBUFSIZE;

        return churn_input(pRec, eMode, &tempread);

    }
@@ -462,10 +462,9 @@ static apr_status_t ssl_io_filter_Input(ap_filter_t *f, 
        return APR_SUCCESS;

    }

   

    /* Readbytes == 0 implies we only want a LF line. 

     * 1024 seems like a good number for now. */

    /* Readbytes == 0 implies we only want a LF line. */

    if (APR_BRIGADE_EMPTY(ctx->b)) {

        tempread = 1024; 

        tempread = AP_IOBUFSIZE;

        rv = churn_input(ctx, mode, &tempread);

        if (rv != APR_SUCCESS)

            return rv;
@@ -505,7 +504,7 @@ static apr_status_t ssl_io_filter_Input(ap_filter_t *f, 


        /* Hey, we're about to be starved - go fetch more data. */

        if (APR_BRIGADE_EMPTY(ctx->b)) {

            tempread = 1024;

            tempread = AP_IOBUFSIZE;

            ret = churn_input(ctx, mode, &tempread);

            if (ret != APR_SUCCESS)

	            return ret;