Closed Bug 5427 Opened 26 years ago Closed 25 years ago

Passing security principals into nsILiveConnect methods will cause browser to crash

Categories

(Core Graveyard :: Java: OJI, defect, P3)

x86
Windows NT

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: stanley.ho, Assigned: drapeau)

Details

Attachments

(1 file)

When an applet called back from Java to JS through netscape.javascript.JSObject, nsILiveConnect will be used. However, when passing any security principals (nsIPrincipal) interface into nsILiveConnect to make any call, it will cause the viewer.exe to crash immediately. This makes even to support the minimial codebase principal impossible.
Can you give me a URL? Also, do I need to get a more recent build than that found in the latest installer from you (Stanley) in order to repro this?
Status: NEW → ASSIGNED
Can you give me a URL? Also, do I need to get a more recent build than that found in the latest installer from you (Stanley) in order to repro this?
Target Milestone: M7
Reassigning all OJI bugs to george.drapeau@eng.sun.com
Status: NEW → ASSIGNED
adding norris to the cc list. this looks like a bug that might need to be move a out a milestone or two. is the problem/fix understood yet?
Target Milestone: M9 → M10
Yes, Chris, I agree: I am moving this to the M10 milestone as a start. Ed Burns (edburns@acm.org) is getting up to speed on OJI for Win32 to try and understand this problem; when he gets ready, I'll probably re-assign the bug to him.
Target Milestone: M10 → M11
moving to m11. we are trying wrap up the m10 list. is a fix for this close?
I haven't worked with OJI either; I probably broke the connection with some of my recent changes. I removed a ton of old code that used to connect from caps to Mozilla classic and was dead. I haven't begun to worry about Java yet. Perhaps we should meet to begin understanding what we need to do here.
Can someone please code review this patch? Ed
Target Milestone: M11 → M12
m12. ed, who might a be a good reviewer? maybe beard or warren?
I just reviewed it. The change looks good to me. R = marklin
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
Fix checked in. I claim this bug is fixed.
Verified per programmer's comments
Status: RESOLVED → VERIFIED
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: