https://issues.apache.org/bugzilla/show_bug.cgi?id=56924

--- Comment #4 from Mark Montague <[email protected]> ---
(In reply to jkaluza from comment #2)
> I'm going to apply this patch to trunk, but I think the second part of the
> patch is not needed. [...]
> This just copies '\0', so everything looks OK. Do you agree or there was
> something I'm missing here?

I agree that the second part of the patch is not needed and does not affect
anything.

My concern was that if the original value for w was "ContentType:\0" then the
'\0' that got copied would be the one that replaced the ':' rather than the one
immediately after it.  This is purely an academic point.

I won't be able to check whether w is actually w is "ContentType:\n\n\0" or
"ContentType:\0" until tonight.

Thanks for the quick and helpful handling of this bug!

-- 
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