Closed Bug 33798 Opened 24 years ago Closed 23 years ago
Overlays can be loaded in an infinite recursion (or cycle)
Overview Description: In the category of boring, defensive programming ... An overlay can load itself as an overlay. Perhaps more critically is that an overlay 'A' may load an overlay 'B' which loads overlay 'A' ... which may be a more possible use case as packages and package extensions are developed. This is of course an error on the part of the package developer, but it is a error condition that could/should be trapped. Steps to Reproduce: 1) Add the line <?xul-overlay href="chrome://navigator/content/netscape.xul"?> to the top of chrome://navigator/content/netscape.xul, and then start mozilla. [Setting up a cycle is, of course, just one additional step away]. Actual Results: infinite loop Expected Results: either: a) abort the launch with an error message (if possible). b) ignore further overlays as soon as the cycle|recursion is detected. Reproducibility: always Build Date & Platform Bug Found: 2000032709 win98 (although I didn't "find" this one.) Additional Builds and Platforms Tested On: this is xp
I think we could live with this for now, moving to siberia & cc'ing hyatt in case he feels otherwise.
Target Milestone: --- → M20
Mass move of all M20 bugs to M30.
Mass move of M20 bugs to M30
Target Milestone: M20 → M30
Mass-moving all M20-M30 XPToolkit bugs to Future
Target Milestone: M30 → Future
->hyatt/untargetted. We are getting requests from the Apps group to make their lives easier in ways like this, when/where we can.
Assignee: trudelle → hyatt
Status: ASSIGNED → NEW
Priority: P3 → P4
Target Milestone: Future → ---
this is fixed (by pollmann's fix)
Status: NEW → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
Component: XP Toolkit/Widgets: XUL → XUL
QA Contact: jrgmorrison → xptoolkit.widgets
You need to log in before you can comment on or make changes to this bug.