Closed
Bug 193256
Opened 21 years ago
Closed 21 years ago
Adding any control to blacklist closes mozilla when any active-x content opened
Categories
(Core Graveyard :: Embedding: ActiveX Wrapper, defect)
Tracking
(Not tracked)
VERIFIED
FIXED
mozilla1.3final
People
(Reporter: ashshbhatt, Assigned: adamlock)
References
Details
Attachments
(1 file)
1.69 KB,
patch
|
dbradley
:
review+
alecf
:
superreview+
dbaron
:
approval1.3+
|
Details | Diff | Splinter Review |
Tested with 2003-02-13 Steps to reproduce 1. Add any control to HKEY_LOCAL_MACHINE\SOFTWARE\Mozilla\ActiveX\Blacklist\CLSID 2. Start mozilla 3. Open page with Active-x content
Small patch Bustage caused by a loop that didn't break properly, meaning a counter would spin forever trying to enumerate keys. I also noticed two args in the plugin were mixed up causing controls on the whitelist to be ignored.
Comment on attachment 114432 [details] [diff] [review] Patch Dave & Alec, can you r/sr this simple patch please? It fixes the blacklist/whitelist lookup in the COM connect object & plugin. Thanks
Attachment #114432 -
Flags: superreview?(alecf)
Attachment #114432 -
Flags: review?(dbradley)
Comment 3•21 years ago
|
||
Comment on attachment 114432 [details] [diff] [review] Patch sr=alecf
Attachment #114432 -
Flags: superreview?(alecf) → superreview+
Comment 4•21 years ago
|
||
Comment on attachment 114432 [details] [diff] [review] Patch r=dbradley
Attachment #114432 -
Flags: review?(dbradley) → review+
Comment on attachment 114432 [details] [diff] [review] Patch Seeking 1.3 approval ActiveX / COM connect specific. Low risk obvious fixes.
Attachment #114432 -
Flags: approval1.3?
Attachment #114432 -
Flags: approval1.3? → approval1.3+
Fix is checked in
Status: NEW → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
Updated•21 years ago
|
Target Milestone: --- → mozilla1.3final
Updated•12 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•