> Resubmit this patch, including changes based on comments by various > people. > > - Mention RFC text in relation to changing the default behaviour in > relation > to unknown HTTP methods. > - Use safe_free instead of xfree. > - Rework urlAbsolute to use snprintf in a slightly better way. Snprintf > is now > used to construct the initial portion of the url and the rest is added > on > using POSIX string routines. >
I'm sure you can still crop the xstrdup() usage by adding a JIT allocation of urlbuf before "if (req->protocol == PROTO_URN)" and returning urlbuf plain at the end. Amos