empty trust_root is no trust root.

work around some thoughtful and considerate RPs that take the pain of sending
the trust_root parameter, but do not go into as much trouble as setting it to
anything but an empty string.

Signed-off-by: Michael Krelin <hacker@klever.net>

1 files changed, 6 insertions, 0 deletions

diff --git a/lib/basic_op.cc b/lib/basic_op.cc
index fa659ac..511b51a 100644
--- a/lib/basic_op.cc
+++ b/lib/basic_op.cc
@@ -149,25 +149,31 @@ namespace opkele {
 	try {
 	    return_to = inm.get_field("return_to");
 	}catch(failed_lookup&) { }
 	if(openid2) {
 	    try {
 		realm = inm.get_field("realm");
+		if(realm.empty())
+		    throw failed_lookup(OPKELE_CP_ "Empty realm doesn't count");
 	    }catch(failed_lookup&) {
 		try {
 		    realm = inm.get_field("trust_root");
+		    if(realm.empty())
+			throw failed_lookup(OPKELE_CP_ "Empty trust_root doesn't count");
 		}catch(failed_lookup&) {
 		    if(return_to.empty())
 			throw bad_input(OPKELE_CP_
 				"Both realm and return_to are unset");
 		    realm = return_to;
 		}
 	    }
 	}else{
 	    try {
 		realm = inm.get_field("trust_root");
+		if(realm.empty())
+		    throw failed_lookup(OPKELE_CP_ "Empty trust_root doesn't count");
 	    }catch(failed_lookup&) {
 		if(return_to.empty())
 		    throw bad_input(OPKELE_CP_
 			    "Both realm and return_to are unset");
 		realm = return_to;
 	    }