I knew it: It's always best to think about a patch one hour more. After I
again and again read the patch (because I wanted to make sure we don't break anything just before release), I find out: _both_ ap_snprintf and ap_psprintf are wrong in the context. Because "args" is a va_list and _CANNOT_ be used safely for ap_snprintf and therefore not for ap_psprintf, too. But (puhhhh) we have ap_vsnprintf in our libap, so use this... So, we fix two things here: 1. The logic of the if-clause 2. The string creating itself git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/branches/1.3.x@82118 13f79535-47bb-0310-9956-ffa450edef68
Loading
Please sign in to comment