[cpp-sp] branch master updated: SSPCPP-754 Invert logic for 32 bit safe integers

[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=2935cd1a16e633897acf3e0001409315bcaa8d0f

The following commit(s) were added to refs/heads/master by this push:
       new  2935cd1   SSPCPP-754 Invert logic for 32 bit safe integers
2935cd1 is described below

commit 2935cd1a16e633897acf3e0001409315bcaa8d0f
Author: Rod Widdowson <rdw at steadingsoftware.com>
AuthorDate: Sun Oct 29 15:44:56 2017 +0000

    SSPCPP-754 Invert logic for 32 bit safe integers
    
    https://issues.shibboleth.net/jira/browse/SSPCPP-754
---
 shibsp/remoting/impl/ddf.cpp | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/shibsp/remoting/impl/ddf.cpp b/shibsp/remoting/impl/ddf.cpp
index 7087aa6..64a63b3 100644
--- a/shibsp/remoting/impl/ddf.cpp
+++ b/shibsp/remoting/impl/ddf.cpp
@@ -866,7 +866,8 @@ void serialize(ddf_body_t* p, ostream& os, bool name_attr=true)
                 break;
 
             case ddf_body_t::DDF_INT:
-                if (is32bitSafe(p->value.integer)) throw IOException("Integer Overflow");
+                if (!is32bitSafe(p->value.integer))
+                    throw IOException("Integer Overflow");
                 os << "<number";
                 if (name_attr && p->name) {
                     os << " name=\"";

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