Infrastructure/caddy
1
Fork 0
mirror of https://github.com/caddyserver/caddy.git synced 2024-12-16 21:56:40 -05:00
Code Issues Activity

Added a null check to avoid segfault on rewrite query ops (#6191)

Browse source
This commit is contained in:
Aziz Rmadi 2024-03-23 00:51:34 -05:00 committed by GitHub
parent 29f57faa86
commit db9d167354
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 7 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

12
modules/caddyhttp/rewrite/rewrite.go
View file

@ -117,13 +117,15 @@ func (rewr *Rewrite) Provision(ctx caddy.Context) error {
}
rep.re = re
}
for _, replacementOp := range rewr.Query.Replace {
err := replacementOp.Provision(ctx)
if err != nil {
return fmt.Errorf("compiling regular expression %s in query rewrite replace operation: %v", replacementOp.SearchRegexp, err)
if rewr.Query != nil {
for _, replacementOp := range rewr.Query.Replace {
err := replacementOp.Provision(ctx)
if err != nil {
return fmt.Errorf("compiling regular expression %s in query rewrite replace operation: %v", replacementOp.SearchRegexp, err)
}
}
}
return nil
}