correct_escapes doesn't work when matching \ #912

Closed
qnet-herwin opened this Issue Feb 27, 2015 · 0 comments

Projects

None yet

1 participant

@qnet-herwin
Contributor

Tested with 3.0.7 and 3.0.x (HEAD: commit a510e19). Easiest way to test is to add deny_realms to the authorize section of sites-enabled/default. This policy does this:

deny_realms {
  if (&User-Name && (&User-Name =~ /@|\\/)) {
    reject
  }
}

Trying to start the server now generates an error:

Error: etc/raddb/policy.d/filter[6]: Invalid regular expression:
Error: etc/raddb/policy.d/filter[6]: @|\
Error: etc/raddb/policy.d/filter[6]:    ^ Pattern compilation failed: \ at end of pattern

Replacing \\ with \\\\ does work, but this is not really what is to be expected with correct_escapes enabled.

@alandekok alandekok closed this in a03713f Feb 27, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment