Bug 4504 - HTML rendering causes hang, fix included
Summary: HTML rendering causes hang, fix included
Status: CLOSED DUPLICATE of bug 4312
Alias: None
Product: Claws Mail (GTK 2)
Classification: Unclassified
Component: Plugins/LiteHTML Viewer (show other bugs)
Version: 3.17.8
Hardware: PC Linux
: P3 normal
Assignee: users
URL:
Depends on:
Blocks:
 
Reported: 2021-07-14 04:25 UTC by Frank Mueller
Modified: 2021-08-03 13:20 UTC (History)
0 users

See Also:


Attachments
sample email causing problems, trace and patch (20.50 KB, application/zip)
2021-07-14 04:25 UTC, Frank Mueller
no flags Details

Description Frank Mueller 2021-07-14 04:25:28 UTC
Created attachment 2223 [details]
sample email causing problems, trace and patch

100% utilization on certain emails when HTML rendering w/ litehtml viewer, track back and quick and dirty patch attached, confirmed to work with attached sample email.

Problem in a nutshell: max_width is negative, causes inifinite recursive calls via 4 calls. Fix: return 0 in method below when max_width is negative.

#73 0x00007f73f4329a0f in litehtml::html_tag::render_box(int, int, int, bool) (this=<optimized out>, x=0, y=0, max_width=-1, second_pass=false) at html_tag.cpp:4026
Comment 1 Paul 2021-07-14 15:43:05 UTC

*** This bug has been marked as a duplicate of bug 4312 ***
Comment 2 Frank Mueller 2021-08-01 22:39:42 UTC
Has anyone read this besides marking cross posts?
The report provides an answer, please apply the patch to the repo.
As of now, nothing has been done to fix it, even though a fix exists.
Comment 3 Paul 2021-08-03 13:20:44 UTC
Relax, Frank.

This had to be read in order to mark it as a duplicate. The other came first, so this is the duplicate, even if you have attached a patch. The patch, ideally, should have been attached to #4312.

The patch has now been applied in git. Thanks!

Note You need to log in before you can comment on or make changes to this bug.