author | Michael Krelin <hacker@klever.net> | 2008-02-15 23:21:05 (UTC) |
---|---|---|
committer | Michael Krelin <hacker@klever.net> | 2008-02-15 23:21:05 (UTC) |
commit | ccdfc6eacec435a59d773127762ad0b6bce07149 (patch) (side-by-side diff) | |
tree | 0a653ff550db04a65f6bbda418725c6c966d294f | |
parent | 2048be862b8619ddec90163af05e6472bf5175e4 (diff) | |
download | libopkele-ccdfc6eacec435a59d773127762ad0b6bce07149.zip libopkele-ccdfc6eacec435a59d773127762ad0b6bce07149.tar.gz libopkele-ccdfc6eacec435a59d773127762ad0b6bce07149.tar.bz2 |
small fix to openid message find_ns method
it foolishly failed in presence of ns field and absence of namespace in
question.
Signed-off-by: Michael Krelin <hacker@klever.net>
-rw-r--r-- | lib/openid_message.cc | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/lib/openid_message.cc b/lib/openid_message.cc index 588bd62..75e59b3 100644 --- a/lib/openid_message.cc +++ b/lib/openid_message.cc @@ -204,27 +204,29 @@ namespace opkele { fields,fnc,fields.size()-fnc); else signeds.append(fields,fnc,fields.size()-fnc); } } }catch(failed_lookup&) { signeds.assign(fields,fnc,fields.size()-fnc); } set_field("signed",signeds); } string basic_openid_message::find_ns(const string& uri,const char *pfx) const { - if(has_field("ns")) + try { return get_ns(uri); - return pfx; + }catch(failed_lookup&) { + return pfx; + } } string basic_openid_message::allocate_ns(const string& uri,const char *pfx) { if(!has_field("ns")) return pfx; if(has_ns(uri)) throw bad_input(OPKELE_CP_ "OpenID message already contains namespace"); string rv = pfx; if(has_field("ns."+rv)) { string::reference c=rv[rv.length()]; for(c='a';c<='z' && has_field("ns."+rv);++c); if(c=='z') throw exception(OPKELE_CP_ "Failed to allocate namespace"); |