[cpp-sp COMMIT] in /branches/REL_2: apache/mod_shib.cpp fastcgi/shibauthorizer.cpp fastcgi/shibresponder.cpp isapi_sh...

[noreply at shibboleth.net]

Author: scantor
Date: Thu Jan  8 22:39:44 2015
New Revision: 3889

URL: http://svn.shibboleth.net/view/cpp-sp?rev=3889&view=rev
Log:
https://issues.shibboleth.net/jira/browse/CPPXT-97

Modified:
    branches/REL_2/apache/mod_shib.cpp
    branches/REL_2/fastcgi/shibauthorizer.cpp
    branches/REL_2/fastcgi/shibresponder.cpp
    branches/REL_2/isapi_shib/isapi_shib.cpp
    branches/REL_2/nsapi_shib/nsapi_shib.cpp

Modified: branches/REL_2/apache/mod_shib.cpp
URL: http://svn.shibboleth.net/view/cpp-sp/branches/REL_2/apache/mod_shib.cpp?rev=3889&r1=3888&r2=3889&view=diff
==============================================================================
--- branches/REL_2/apache/mod_shib.cpp (original)
+++ branches/REL_2/apache/mod_shib.cpp Thu Jan  8 22:39:44 2015
@@ -602,19 +602,23 @@
       m_req->content_type = ap_psprintf(m_req->pool, "%s", type);
   }
   void setResponseHeader(const char* name, const char* value) {
-   HTTPResponse::setResponseHeader(name, value);
+    HTTPResponse::setResponseHeader(name, value);
+    if (name) {
 #ifdef SHIB_DEFERRED_HEADERS
-   if (!m_rc)
-      // this happens on subrequests
-      m_rc = get_request_config(m_req);
-    if (m_handler) {
-        if (!m_rc->hdr_out)
-            m_rc->hdr_out = ap_make_table(m_req->pool, 5);
-        ap_table_add(m_rc->hdr_out, name, value);
-    }
-    else
-#endif
-    ap_table_add(m_req->err_headers_out, name, value);
+        if (!m_rc) {
+            // this happens on subrequests
+            m_rc = get_request_config(m_req);
+        }
+        if (m_handler) {
+            if (!m_rc->hdr_out) {
+                m_rc->hdr_out = ap_make_table(m_req->pool, 5);
+            }
+            ap_table_add(m_rc->hdr_out, name, value);
+        }
+        else
+#endif
+            ap_table_add(m_req->err_headers_out, name, value);
+    }
   }
   long sendResponse(istream& in, long status) {
     if (status != XMLTOOLING_HTTP_STATUS_OK)

Modified: branches/REL_2/fastcgi/shibauthorizer.cpp
URL: http://svn.shibboleth.net/view/cpp-sp/branches/REL_2/fastcgi/shibauthorizer.cpp?rev=3889&r1=3888&r2=3889&view=diff
==============================================================================
--- branches/REL_2/fastcgi/shibauthorizer.cpp (original)
+++ branches/REL_2/fastcgi/shibauthorizer.cpp Thu Jan  8 22:39:44 2015
@@ -198,11 +198,13 @@
     }
     void setResponseHeader(const char* name, const char* value) {
         HTTPResponse::setResponseHeader(name, value);
-        // Set for later.
-        if (value)
-            m_response_headers.insert(make_pair(name,value));
-        else
-            m_response_headers.erase(name);
+        if (name) {
+            // Set for later.
+            if (value)
+                m_response_headers.insert(make_pair(name,value));
+            else
+                m_response_headers.erase(name);
+        }
     }
     const char* getQueryString() const {
         return FCGX_GetParam("QUERY_STRING", m_req->envp);

Modified: branches/REL_2/fastcgi/shibresponder.cpp
URL: http://svn.shibboleth.net/view/cpp-sp/branches/REL_2/fastcgi/shibresponder.cpp?rev=3889&r1=3888&r2=3889&view=diff
==============================================================================
--- branches/REL_2/fastcgi/shibresponder.cpp (original)
+++ branches/REL_2/fastcgi/shibresponder.cpp Thu Jan  8 22:39:44 2015
@@ -150,11 +150,13 @@
 
     void setResponseHeader(const char* name, const char* value) {
         HTTPResponse::setResponseHeader(name, value);
-        // Set for later.
-        if (value)
-            m_headers.insert(make_pair(name,value));
-        else
-            m_headers.erase(name);
+        if (name) {
+            // Set for later.
+            if (value)
+                m_headers.insert(make_pair(name,value));
+            else
+                m_headers.erase(name);
+        }
     }
 
     const char* getQueryString() const {

Modified: branches/REL_2/isapi_shib/isapi_shib.cpp
URL: http://svn.shibboleth.net/view/cpp-sp/branches/REL_2/isapi_shib/isapi_shib.cpp?rev=3889&r1=3888&r2=3889&view=diff
==============================================================================
--- branches/REL_2/isapi_shib/isapi_shib.cpp (original)
+++ branches/REL_2/isapi_shib/isapi_shib.cpp Thu Jan  8 22:39:44 2015
@@ -492,11 +492,13 @@
   }
   void setResponseHeader(const char* name, const char* value) {
     HTTPResponse::setResponseHeader(name, value);
-    // Set for later.
-    if (value)
-        m_headers.insert(make_pair(name,value));
-    else
-        m_headers.erase(name);
+    if (name) {
+        // Set for later.
+        if (value)
+            m_headers.insert(make_pair(name,value));
+        else
+            m_headers.erase(name);
+    }
   }
   long sendResponse(istream& in, long status) {
     string hdr = string("Connection: close\r\n");
@@ -866,11 +868,13 @@
   }
   void setResponseHeader(const char* name, const char* value) {
     HTTPResponse::setResponseHeader(name, value);
-    // Set for later.
-    if (value)
-        m_headers.insert(make_pair(name,value));
-    else
-        m_headers.erase(name);
+    if (name) {

[... 27 lines stripped ...]