diff options
author | Ashish Gupta <ashmew2@gmail.com> | 2017-05-11 17:14:01 +0200 |
---|---|---|
committer | Ashish Gupta <ashmew2@gmail.com> | 2017-06-10 08:25:29 +0200 |
commit | 839c2e8e6342dae087b73812ffeffda47af2094a (patch) | |
tree | e5ff3a7bc8b2e4bf6364cfb4508b3e383b1213c5 /content/fetchers/httplib_kolibri.c | |
parent | 5aeb8eb5583d7d088c8132d3dca0dc3c34d8c565 (diff) | |
download | netsurf-839c2e8e6342dae087b73812ffeffda47af2094a.tar.gz netsurf-839c2e8e6342dae087b73812ffeffda47af2094a.tar.bz2 |
Tidy up some things.
Diffstat (limited to 'content/fetchers/httplib_kolibri.c')
-rw-r--r-- | content/fetchers/httplib_kolibri.c | 29 |
1 files changed, 16 insertions, 13 deletions
diff --git a/content/fetchers/httplib_kolibri.c b/content/fetchers/httplib_kolibri.c index 0f9bcb7c4..6800cab13 100644 --- a/content/fetchers/httplib_kolibri.c +++ b/content/fetchers/httplib_kolibri.c @@ -85,16 +85,16 @@ void remove_from_poll(struct http_msg *donehttp) { bool init_fetcher(lwc_string *scheme) { bool supported_scheme; assert(lwc_string_isequal(scheme, corestring_lwc_http, &supported_scheme) == lwc_error_ok); - kolibri_http_init(); - /* if(supported_scheme) { */ - /* if(kolibri_http_init() == 0) */ - /* debug_board_printf("[INFO] Loaded http.obj library successfully.\n"); */ - /* else { */ - /* debug_board_printf("[ERROR] Could not load http.obj library.\n"); */ - /* return false; */ - /* } */ - /* } */ + LOG("Initializing http library!"); + debug_board_printf("fetcher_init: Initializing http library!\n"); + if(kolibri_http_init() == 0) + LOG("[INFO] Loaded http.obj library successfully.\n"); + else { + LOG("[ERROR] Could not load http.obj library.\n"); + assert(0 && 1); + return false; + } return supported_scheme; } @@ -138,14 +138,17 @@ void *setup_fetch(struct fetch *parent_fetch, struct nsurl *url, request = http_get_asm(nsurl_access(url), NULL, 0, *headers); } - LOG("Address of request : 0x%x", request); - assert(request != NULL); - if(request == NULL) { - LOG("[ERROR] Cannot allocate more memory for http library handle creation."); + LOG("Failed to allocate http buffer. Could be multiple reasons for failure (like DNS resolution)", request); return NULL; } + /* This assert is tricky beacause the http library may fail when it fails to resolve the hostname. + There is no sane way to understand that the hostname is wrong or needs to be fixed. + assert leads to a crash for no reason. Disabling it, and the code should handle this from now. + Returning NuLL from here should just force a BadURL to the user. + */ + struct httpfetcher *newfetcher = (struct httpfetcher *) malloc(sizeof(struct httpfetcher)); assert(newfetcher); |