[cpp-sp] branch master updated: SSPCPP-726 Fix condition's scoping

[Rod Widdowson]

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

rdw pushed a commit to branch master
in repository cpp-sp.

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

The following commit(s) were added to refs/heads/master by this push:
       new  5dcb0ee   SSPCPP-726 Fix condition's scoping
5dcb0ee is described below

commit 5dcb0eef4bc42bffc73b91a79a4395087469532d
Author: Rod Widdowson <rdw at steadingsoftware.com>
AuthorDate: Tue Feb 14 11:35:34 2017 +0000

    SSPCPP-726 Fix condition's scoping
    
    https://issues.shibboleth.net/jira/browse/SSPCPP-726
---
 shibsp/handler/impl/ExternalAuthHandler.cpp | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/shibsp/handler/impl/ExternalAuthHandler.cpp b/shibsp/handler/impl/ExternalAuthHandler.cpp
index c17ca1f..e6faca9 100644
--- a/shibsp/handler/impl/ExternalAuthHandler.cpp
+++ b/shibsp/handler/impl/ExternalAuthHandler.cpp
@@ -592,8 +592,10 @@ pair<bool,long> ExternalAuth::processMessage(
             }
         }
         os << " ]";
-        if (!target.empty())
-            os << ", \"RelayState\": "; json_safe(os, target.c_str());
+        if (!target.empty()) {
+            os << ", \"RelayState\": ";
+            json_safe(os, target.c_str());
+        }
         os << " }";
     }
     else {

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