[cpp-sp] branch main updated: SSPCPP-937 - RequestMap doesn't reset internal variable during path decode

[Scott Cantor]

This is an automated email from the git hooks/post-receive script.

scantor pushed a commit to branch main
in repository cpp-sp.

View the commit online:
http://git.shibboleth.net/view/?p=cpp-sp.git;a=commit;h=75b0b30ccf8d7befee8bf6437991d7ad5acf074e

The following commit(s) were added to refs/heads/main by this push:
       new  75b0b30c  SSPCPP-937 - RequestMap doesn't reset internal variable during path decode
75b0b30c is described below

commit 75b0b30ccf8d7befee8bf6437991d7ad5acf074e
Author: Scott Cantor <cantor.2 at osu.edu>
AuthorDate: Fri Jul 2 13:58:39 2021 -0400

    SSPCPP-937 - RequestMap doesn't reset internal variable during path decode
    
    https://issues.shibboleth.net/jira/browse/SSPCPP-937
---
 shibsp/impl/XMLRequestMapper.cpp | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/shibsp/impl/XMLRequestMapper.cpp b/shibsp/impl/XMLRequestMapper.cpp
index 95ec2ec7..36271784 100644
--- a/shibsp/impl/XMLRequestMapper.cpp
+++ b/shibsp/impl/XMLRequestMapper.cpp
@@ -419,6 +419,9 @@ const Override* Override::locate(const HTTPRequest& request) const
     // Default is for the current object to provide settings.
     const Override* o = this;
 
+    // Reset the path pointer to the beginning of the decoded copy.
+    path = dup.c_str();
+
     // Tokenize the path by segment and try and map each segment.
     tokenizer< char_separator<char> > tokens(dup, char_separator<char>("/"));
     for (tokenizer< char_separator<char> >::iterator token = tokens.begin(); token != tokens.end(); ++token) {

-- 
To stop receiving notification emails like this one, please contact
the administrator of this repository.