https://bz.apache.org/bugzilla/show_bug.cgi?id=69975

--- Comment #1 from Dan Anderson <[email protected]> ---
My suggested fix would be to rewrite the function as follows:

static const char *set_limit_xml_req_body(cmd_parms *cmd, void *conf_,
                                          const char *arg)
{
    core_dir_config *conf = conf_;
    char *errp;
    apr_off_t limit;

    if (APR_SUCCESS != apr_strtoff(&limit, arg, &errp, 10)) {
        return "LimitXMLRequestBody argument is not parsable.";
    }
    if (*errp || limit < 0) {
        return "LimitXMLRequestBody requires a non-negative integer.";
    }
    conf->limit_xml_body = (unsigned long)limit;

    /* zero is AP_MAX_LIMIT_XML_BODY (implicitly) */
    if ((apr_size_t)conf->limit_xml_body > AP_MAX_LIMIT_XML_BODY)
        return apr_psprintf(cmd->pool, "LimitXMLRequestBody must not exceed "
                            "%" APR_SIZE_T_FMT, AP_MAX_LIMIT_XML_BODY);

    return NULL;
}

-- 
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to