diff options
author | Daniel Silverstone <dsilvers@digital-scurf.org> | 2019-08-23 21:58:07 +0100 |
---|---|---|
committer | Daniel Silverstone <dsilvers@digital-scurf.org> | 2019-08-23 21:58:07 +0100 |
commit | 524965b867e9aaef4ba75d6e077fdfd6c0b88401 (patch) | |
tree | ef8a9d4b25e7131a9a793338ec3cd896c3760e70 /content/handlers/html/html.c | |
parent | cb33f696a2683e7e848a0444f52945f2f56acc28 (diff) | |
download | netsurf-524965b867e9aaef4ba75d6e077fdfd6c0b88401.tar.gz netsurf-524965b867e9aaef4ba75d6e077fdfd6c0b88401.tar.bz2 |
Box Conversion: Cancel conversion during html_destroy
If dom_to_box is still in progress when we destroy an HTML
content, we need to cancel the conversion otherwise we will
end up with a scheduled callback into infinity.
Signed-off-by: Daniel Silverstone <dsilvers@digital-scurf.org>
Diffstat (limited to 'content/handlers/html/html.c')
-rw-r--r-- | content/handlers/html/html.c | 11 |
1 files changed, 10 insertions, 1 deletions
diff --git a/content/handlers/html/html.c b/content/handlers/html/html.c index ed48af20e..33dcf2aaa 100644 --- a/content/handlers/html/html.c +++ b/content/handlers/html/html.c @@ -115,6 +115,8 @@ static void html_box_convert_done(html_content *c, bool success) NSLOG(netsurf, INFO, "Done XML to box (%p)", c); + c->box_conversion_context = NULL; + /* Clean up and report error if unsuccessful or aborted */ if ((success == false) || (c->aborted)) { html_object_free_objects(c); @@ -656,7 +658,7 @@ void html_finish_conversion(html_content *htmlc) html_get_dimensions(htmlc); - error = dom_to_box(html, htmlc, html_box_convert_done); + error = dom_to_box(html, htmlc, html_box_convert_done, &htmlc->box_conversion_context); if (error != NSERROR_OK) { NSLOG(netsurf, INFO, "box conversion failed"); dom_node_unref(html); @@ -1710,6 +1712,13 @@ static void html_destroy(struct content *c) NSLOG(netsurf, INFO, "content %p", c); + /* If we're still converting a layout, cancel it */ + if (html->box_conversion_context != NULL) { + if (cancel_dom_to_box(html->box_conversion_context) != NSERROR_OK) { + NSLOG(netsurf, CRITICAL, "WARNING, Unable to cancel conversion context, browser may crash"); + } + } + /* Destroy forms */ for (f = html->forms; f != NULL; f = g) { g = f->prev; |