Lines 726-730 of mod_isapi.c read: newstat = apr_palloc(cid->r->pool, statlen + 9); strcpy(newstat, "Status: "); apr_cpystrn(newstat + 8, stat, statlen); stat = newstat; statlen += 8; Because apr_cpystrn() reserves a byte for a terminal null and statlen is the length of the status string to be copied, the final character of the status string is replaced with a null when it is copied to newstat. The apr_palloc() call allocates sufficient space for the entire status string plus a terminal null, so I think the only change required is in line 728: apr_cpystrn(newstat + 8, stat, statlen + 1);
Created attachment 6741 [details] Trivial patch
*** Bug 21302 has been marked as a duplicate of this bug. ***
enabling the PatchAvailable keyword updated doc on submitting patches is at http://httpd.apache.org/dev/patches.html
thanks for the patch! committed to 2.1-dev, will suggest shortly that it be merged into the stable branch
+1 on purusing the patch. I believe(d) that we handled this header using the statlen bytecount, so as a counted string the trailing null was not important. But trusting your patch :) Thank you.