Bug 24181 - GTK+ crashes in fast/loader/empty-embed-src-attribute.html
Summary: GTK+ crashes in fast/loader/empty-embed-src-attribute.html
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKitGTK (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC All
: P2 Critical
Assignee: Gustavo Noronha (kov)
URL:
Keywords: Gtk
Depends on:
Blocks:
 
Reported: 2009-02-25 19:37 PST by Gustavo Noronha (kov)
Modified: 2009-02-27 18:22 PST (History)
0 users

See Also:


Attachments
proposed fix (2.61 KB, patch)
2009-02-25 22:23 PST, Gustavo Noronha (kov)
no flags Details | Formatted Diff | Diff
a different approach that still loads (2.76 KB, patch)
2009-02-26 04:48 PST, Gustavo Noronha (kov)
no flags Details | Formatted Diff | Diff
yet another approach (3.26 KB, patch)
2009-02-26 16:16 PST, Gustavo Noronha (kov)
hyatt: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Gustavo Noronha (kov) 2009-02-25 19:37:37 PST
This is because a load is tried for a null request, which ends up crashing. This is most probably caused by the load code paths not doing proper checks or conversions on the URL or the request. The crash doesn't happen in other ports because the URL gets set to something by the port's platform code.

See https://bugs.webkit.org/show_bug.cgi?id=23806 for more information.
Comment 1 Gustavo Noronha (kov) 2009-02-25 22:23:31 PST
Created attachment 28005 [details]
proposed fix

As I noted on the bug report I mentioned when reporting this one, this doesn't seem to be a GTK+-specific problem, and this is probably not the definitive fix, but since this test was added with a work-around fix for other ports, I believe this work-around for GTK+ is in order.
Comment 2 Gustavo Noronha (kov) 2009-02-26 04:48:49 PST
Created attachment 28012 [details]
a different approach that still loads

This is a better approach that still causes the load events to happen, as intended.
Comment 3 Holger Freyther 2009-02-26 07:31:36 PST
That looks so ugly. Give me some hours to find something better.
Comment 4 Holger Freyther 2009-02-26 10:10:59 PST
19:04 < dglazkov> zecke: I can't see this problem. My brain has completely blacklisted anything that may even potentially be related to this 
                  issue
19:04 < dglazkov> :)
19:05 < dglazkov> zecke: but somewhere deep in the back, it's still thinking about it!
19:05 < dglazkov> I think it's in medula oblongata
19:05 < zecke> dglazkov: hehe, the question is what is chromium or the mac loading for an empty url?
19:07 < dglazkov> the policy checker issues PolicyIgnore for empty URLs
19:07 < zecke> thanks! :)
19:07 < zecke> _manyoso: you want to see the above as well
19:07 < dglazkov> zecke, _manyoso: but that's the wrong fix.
19:08 < dglazkov> we should really just fix it in the loader
19:08 < dglazkov> I mean, it's an okay short-term fix

so PolicyIgnore would be another option?
Comment 5 Gustavo Noronha (kov) 2009-02-26 16:16:47 PST
Created attachment 28051 [details]
yet another approach

OK, this works too =).
Comment 6 Dave Hyatt 2009-02-27 16:42:01 PST
Comment on attachment 28051 [details]
yet another approach

r=me
Comment 7 Gustavo Noronha (kov) 2009-02-27 18:22:35 PST
Landed in r41313.