Anything which eventually has the value it got interpreted by pairparsevalue, or...
authorArran Cudbard-Bell <a.cudbardb@freeradius.org>
Wed, 7 May 2014 15:14:40 +0000 (16:14 +0100)
committerArran Cudbard-Bell <a.cudbardb@freeradius.org>
Wed, 7 May 2014 15:17:54 +0000 (16:17 +0100)
commit4471ce098982f10384bb041801f864ef59b884da
tree803b3893ca1704b9e0fb6705846dca7315aec5c1
parent5d6c4e800e3b425492ce5791cb1783efdc10327e
Anything which eventually has the value it got interpreted by pairparsevalue, or doesn't pass the length of the string around must not pre-unescape the string

This broke pretty much all escape sequences in preprocess, sql, ldap and radclient. Arguably as conffile.c is the only place which needs this, it should probably be the thing doing the unescaping.

No, I don't buy that things should be prefixed with \\, that's shit. The server should be able to reingest what it spits out in detail files and there's no way that worked in the current server with octal escape sequences.
src/include/token.h
src/lib/token.c
src/lib/valuepair.c
src/main/conffile.c
src/main/files.c
src/modules/rlm_preprocess/rlm_preprocess.c
src/modules/rlm_sql/sql.c