On 17.04.2013 11:07, André Malo wrote: > * André Malo wrote: > >> * Rainer Jung wrote: >>> A quick draft patch is at >>> >>> http://people.apache.org/~rjung/patches/ap_escape_logitem_enhanced.patc >>> h >> >> + ret = apr_palloc(p, length + 3 * escapes); >> >> It should be still 4 * escapes ('\xHH').
Hmmm, isn't the 1 = 4-3 already counted in length? Here we only add the escaping overhead of 3 bytes per escape. > Also, escapes should be apr_size_t as well, I think. Probably. Thanks, Rainer
