-rw-r--r-- | include/opkele/expat.h | 6 | ||||
-rw-r--r-- | lib/discovery.cc | 18 |
2 files changed, 16 insertions, 8 deletions
diff --git a/include/opkele/expat.h b/include/opkele/expat.h index 60c41ac..97ed61a 100644 --- a/include/opkele/expat.h +++ b/include/opkele/expat.h | |||
@@ -20,14 +20,16 @@ namespace opkele { | |||
20 | 20 | ||
21 | operator const XML_Parser(void) const { return _x; } | 21 | operator const XML_Parser(void) const { return _x; } |
22 | operator XML_Parser(void) { return _x; } | 22 | operator XML_Parser(void) { return _x; } |
23 | 23 | ||
24 | inline bool parse(const char *s,int len,bool final=false) { | 24 | inline bool parse(const char *s,int len,bool final=false) { |
25 | assert(_x); | 25 | assert(_x); |
26 | return XML_Parse(_x,s,len,final); | 26 | return XML_Parse(_x,s,len,final); } |
27 | } | 27 | enum XML_Status stop_parser(bool resumable=false) { |
28 | assert(_x); | ||
29 | return XML_StopParser(_x,resumable); } | ||
28 | 30 | ||
29 | virtual void start_element(const XML_Char *n,const XML_Char **a) { } | 31 | virtual void start_element(const XML_Char *n,const XML_Char **a) { } |
30 | virtual void end_element(const XML_Char *n) { } | 32 | virtual void end_element(const XML_Char *n) { } |
31 | void set_element_handler(); | 33 | void set_element_handler(); |
32 | 34 | ||
33 | virtual void character_data(const XML_Char *s,int l) { } | 35 | virtual void character_data(const XML_Char *s,int l) { } |
diff --git a/lib/discovery.cc b/lib/discovery.cc index 81727c0..8729cfb 100644 --- a/lib/discovery.cc +++ b/lib/discovery.cc | |||
@@ -2,12 +2,13 @@ | |||
2 | #include <opkele/curl.h> | 2 | #include <opkele/curl.h> |
3 | #include <opkele/expat.h> | 3 | #include <opkele/expat.h> |
4 | #include <opkele/uris.h> | 4 | #include <opkele/uris.h> |
5 | #include <opkele/discovery.h> | 5 | #include <opkele/discovery.h> |
6 | #include <opkele/exception.h> | 6 | #include <opkele/exception.h> |
7 | #include <opkele/util.h> | 7 | #include <opkele/util.h> |
8 | #include <opkele/debug.h> | ||
8 | 9 | ||
9 | #include "config.h" | 10 | #include "config.h" |
10 | 11 | ||
11 | #define XRDS_HEADER "X-XRDS-Location" | 12 | #define XRDS_HEADER "X-XRDS-Location" |
12 | #define CT_HEADER "Content-Type" | 13 | #define CT_HEADER "Content-Type" |
13 | 14 | ||
@@ -172,13 +173,17 @@ namespace opkele { | |||
172 | } | 173 | } |
173 | 174 | ||
174 | size_t write(void *p,size_t s,size_t nm) { | 175 | size_t write(void *p,size_t s,size_t nm) { |
175 | if(skipping<0) return 0; | 176 | if(skipping<0) return 0; |
176 | /* TODO: limit total size */ | 177 | /* TODO: limit total size */ |
177 | size_t bytes = s*nm; | 178 | size_t bytes = s*nm; |
178 | parse((const char *)p,bytes,false); | 179 | bool rp = parse((const char *)p,bytes,false); |
180 | if(!rp) { | ||
181 | skipping = -1; | ||
182 | bytes = 0; | ||
183 | } | ||
179 | return bytes; | 184 | return bytes; |
180 | } | 185 | } |
181 | size_t header(void *p,size_t s,size_t nm) { | 186 | size_t header(void *p,size_t s,size_t nm) { |
182 | size_t bytes = s*nm; | 187 | size_t bytes = s*nm; |
183 | const char *h = (const char*)p; | 188 | const char *h = (const char*)p; |
184 | const char *colon = (const char*)memchr(p,':',bytes); | 189 | const char *colon = (const char*)memchr(p,':',bytes); |
@@ -222,13 +227,13 @@ namespace opkele { | |||
222 | assert(xrd); | 227 | assert(xrd); |
223 | xrd->clear(); | 228 | xrd->clear(); |
224 | pt_stack.push_back(n); | 229 | pt_stack.push_back(n); |
225 | }else if(xmode&xmode_html) { | 230 | }else if(xmode&xmode_html) { |
226 | html_start_element(n,a); | 231 | html_start_element(n,a); |
227 | }else{ | 232 | }else{ |
228 | skipping = -1; | 233 | skipping = -1; stop_parser(); |
229 | } | 234 | } |
230 | }else{ | 235 | }else{ |
231 | int pt_s = pt_stack.size(); | 236 | int pt_s = pt_stack.size(); |
232 | if(pt_s==1) { | 237 | if(pt_s==1) { |
233 | if(is_qelement(n,NSURI_XRD "\tCanonicalID")) { | 238 | if(is_qelement(n,NSURI_XRD "\tCanonicalID")) { |
234 | assert(xrd); | 239 | assert(xrd); |
@@ -307,20 +312,21 @@ namespace opkele { | |||
307 | xrd_service = 0; | 312 | xrd_service = 0; |
308 | }else if(is_qelement(n,NSURI_XRD "\tStatus")) { | 313 | }else if(is_qelement(n,NSURI_XRD "\tStatus")) { |
309 | assert(xrd); | 314 | assert(xrd); |
310 | if(is_qelement(pt_stack.back().c_str(),n)) { | 315 | if(is_qelement(pt_stack.back().c_str(),n)) { |
311 | assert(cdata==&status_string); | 316 | assert(cdata==&status_string); |
312 | pt_stack.pop_back(); | 317 | pt_stack.pop_back(); |
313 | if(status_code!=100) | 318 | if(status_code!=100) { |
314 | skipping = -1; | 319 | skipping = -1; stop_parser(); |
320 | } | ||
315 | } | 321 | } |
316 | }else if(is_qelement(n,NSURI_XRD "\tExpires")) { | 322 | }else if(is_qelement(n,NSURI_XRD "\tExpires")) { |
317 | assert(xrd); | 323 | assert(xrd); |
318 | xrd->expires = util::w3c_to_time(cdata_buf); | 324 | xrd->expires = util::w3c_to_time(cdata_buf); |
319 | }else if((xmode&xmode_html) && is_element(n,"head")) { | 325 | }else if((xmode&xmode_html) && is_element(n,"head")) { |
320 | skipping = -1; | 326 | skipping = -1; stop_parser(); |
321 | } | 327 | } |
322 | cdata = 0; | 328 | cdata = 0; |
323 | } | 329 | } |
324 | void character_data(const XML_Char *s,int l) { | 330 | void character_data(const XML_Char *s,int l) { |
325 | if(skipping) return; | 331 | if(skipping) return; |
326 | if(cdata) cdata->append(s,l); | 332 | if(cdata) cdata->append(s,l); |
@@ -371,13 +377,13 @@ namespace opkele { | |||
371 | else if(rel=="openid2.provider") | 377 | else if(rel=="openid2.provider") |
372 | html_openid2.uris.add(-1,href); | 378 | html_openid2.uris.add(-1,href); |
373 | else if(rel=="openid2.local_id") | 379 | else if(rel=="openid2.local_id") |
374 | html_openid2.local_ids.add(-1,href); | 380 | html_openid2.local_ids.add(-1,href); |
375 | } | 381 | } |
376 | }else if(is_element(n,"body")) { | 382 | }else if(is_element(n,"body")) { |
377 | skipping = -1; | 383 | skipping = -1; stop_parser(); |
378 | } | 384 | } |
379 | } | 385 | } |
380 | 386 | ||
381 | }; | 387 | }; |
382 | 388 | ||
383 | void idiscover(idiscovery_t& result,const string& identity) { | 389 | void idiscover(idiscovery_t& result,const string& identity) { |