Re: Release 1.0.5?

Nicolas Baradakis <nbk@sitadelle.com> wrote:

Thinking about it, I don't like the idea to make rlm_sql export sql_escape_func. And I'm afraid of linkage issues, too. A links to B. A links to C. That's not rocket science, but will C be able to find a symbol in B on systems with stupid linker?

Uh... no.

I'd like to move sql_escape_func in the server core, in src/main/xlat.c for example. However, I noticed this function uses a global variable 'allowed_chars', so we'd need to bring it in the server core, too.

I don't like that, either. I'd rather just copy the function from rlm_sql to rlm_sqlcounter. Alan DeKok.