JDK-6611345 : SSV cleanups and refactorings
  • Type: Bug
  • Component: deploy
  • Sub-Component: plugin
  • Affected Version: 6u5
  • Priority: P2
  • Status: Closed
  • Resolution: Duplicate
  • OS: windows
  • CPU: generic
  • Submitted: 2007-09-30
  • Updated: 2010-04-04
  • Resolved: 2007-10-31
The Version table provides details related to the release that this issue/RFE will be addressed.

Unresolved : Release in which this issue/RFE will be addressed.
Resolved: Release in which this issue/RFE has been resolved.
Fixed : Release in which this issue/RFE has been fixed. The release containing this fix may be available for download as an Early Access Release or a General Availability Release.

To download the current JDK release, click here.
JDK 6
6u10Resolved
Related Reports
Duplicate :  
Relates :  
Relates :  
Description
The old and new plugins' SSV implementations need to be refactored after the changes for 6581221 are put back. The code now in ssv_common needs to be updated with the modifications made in 6581221, and the code for ssv.dll needs to be changed to use the code in ssv_common.
In particular, the code to delete the old plugins out of the Firefox plugins directory needs to be pulled in, and the GetLoadedModuleDirectory.[ch] just added to the workspace need to be deleted in favor of the version in common.cpp.
The code in the installer needs to be updated to call into the right ssv.dll or jp2ssv.dll to put in place the CLSID registry keys (for the case where we are doing an installation while the browser is still open). This code exists in 6u4, and was just brought forward to 6u5. It is also necessary to do HLKM key redirection as we have found that Vista consults HKLM and not HKCU when UAC is disabled (see 6612714). We will probably need to change the common entry point exposed from ssv.dll and jp2ssv.dll to indicate that the call is being made from a high integrity process (on Vista). We will also likely need to provide the name of the SSV broker executable, or probably more preferably merge the two SSV brokers into one.

Comments
EVALUATION This bug is being marked as a duplicate of 6622332 for bookkeeping purposes and does not need to be re-verified independently. In the case of a regression please file a new bug.
31-10-2007

SUGGESTED FIX http://analemma.sfbay.sun.com/net/jano2/export2/hotspot/users/kbr/webrevs/mjre/6611345_ssv_cleanup/
07-10-2007

EVALUATION Refactored registry modification, old Mozilla plugin deletion, and broker process launching code from SSV sources into SSVCommonUtils. Folded functionality of jp2ssvbroker broker process into ssvagent; made ssvagent capable of working with either old or new plugins' SSV DLL via command-line arguments. Moved ssvagent to top-level source directory instead of being subdirectory of ssv. In installer, made registration and unregistration of ssvagent broker process more general, changed SSVSetup helper function to work with either old or new plugins' SSV DLL, and cleaned up CLSID redirections in HKLM registry branch uniformly instead of only on non-NT or Windows Vista platforms. Tested with installation and uninstallation and applet execution on Windows XP and Vista. *** (#1 of 1): [ UNSAVED ] ###@###.###
07-10-2007