On Wed, Jul 06, 2005 at 11:45:21AM -0500, William Rowe wrote:
...
> +            else {
> +                char *len_end;
> +                errno = 0;
> +                c->len = ap_strtol(content_length, &len_end, 10);
...
> +                if (errno || (c->len < 0) || (len_end && *len_end)) {

You should copy the logic used on the trunk to get the correct tests for 
a strtol parse error:

  errno || len_end == content_length || *len_end || c->len < 0

joe

Reply via email to