https://issues.apache.org/bugzilla/show_bug.cgi?id=56924
--- Comment #2 from [email protected] --- I'm going to apply this patch to trunk, but I think the second part of the patch is not needed. Current code looks like this (shorter version just to show the "l" variable): w is "ContentType:\n\n\0" > l = strchr(w, ':') l is ":\n\n\0" > *l++ = '\0'; l is "\n\n\0" w is "ContentType\0" > while (apr_isspace(*l)) { > ++l; > } l is "\0" (\n\n is skipped) > if (!strcasecmp(w, "Content-type")) { > char *endp = l + strlen(l) - 1; endp is l - 0 - 1. > while (endp > l && apr_isspace(*endp)) { > *endp-- = '\0'; > } This is skipped because enpd < l. > tmp = apr_pstrdup(r->pool, l); This just copies '\0', so everything looks OK. Do you agree or there was something I'm missing here? -- 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]
