/hg/icedtea-web: Fix RH665104: OpenJDK Firefox Java plugin loses...

[dbhole at icedtea.classpath.org]

changeset 70e445bbe85a in /hg/icedtea-web
details: http://icedtea.classpath.org/hg/icedtea-web?cmd=changeset;node=70e445bbe85a
author: Deepak Bhole <dbhole at redhat.com>
date: Wed Dec 22 17:14:44 2010 -0500

	Fix RH665104: OpenJDK Firefox Java plugin loses a cookie


diffstat:

3 files changed, 16 insertions(+), 3 deletions(-)
ChangeLog                                                     |    8 ++++++++
NEWS                                                          |    1 +
plugin/icedteanp/java/sun/applet/PluginCookieInfoRequest.java |   10 +++++++---

diffs (49 lines):

diff -r 7ddab63cf8fe -r 70e445bbe85a ChangeLog
--- a/ChangeLog	Tue Dec 21 16:48:12 2010 -0500
+++ b/ChangeLog	Wed Dec 22 17:14:44 2010 -0500
@@ -1,3 +1,11 @@ 2010-12-21  Andrew Su  <asu at redhat.com>
+2010-12-22  Deepak Bhole <dbhole at redhat.com>
+
+	RH665104: OpenJDK Firefox Java plugin loses a cookie
+	* plugin/icedteanp/java/sun/applet/PluginCookieInfoRequest.java
+	(parseReturn): Skip one less space so that the first cookie is not
+	skipped.
+	* NEWS: Updated.
+
 2010-12-21  Andrew Su  <asu at redhat.com>
 
 	* netx/net/sourceforge/jnlp/controlpanel/AdvancedProxySettingsPane.java,
diff -r 7ddab63cf8fe -r 70e445bbe85a NEWS
--- a/NEWS	Tue Dec 21 16:48:12 2010 -0500
+++ b/NEWS	Wed Dec 22 17:14:44 2010 -0500
@@ -23,6 +23,7 @@ New in release 1.0 (2010-XX-XX):
   - PR593: Increment of invalidated iterator in IcedTeaPluginUtils (patch from barbara.xxx1975 at libero.it)
   - PR597: Entities are parsed incorrectly in PARAM tag in applet plugin
   - Applets are now double-buffered to eliminate flicker in ones that do heavy drawing
+  - RH665104: OpenJDK Firefox Java plugin loses a cookie
 * NetX
   - Add a new option -Xclearcache
   - Interfaces javax.jnlp.IntegrationService and javax.jnlp.DownloadService2 are now available
diff -r 7ddab63cf8fe -r 70e445bbe85a plugin/icedteanp/java/sun/applet/PluginCookieInfoRequest.java
--- a/plugin/icedteanp/java/sun/applet/PluginCookieInfoRequest.java	Tue Dec 21 16:48:12 2010 -0500
+++ b/plugin/icedteanp/java/sun/applet/PluginCookieInfoRequest.java	Wed Dec 22 17:14:44 2010 -0500
@@ -56,12 +56,16 @@ public class PluginCookieInfoRequest ext
 
         PluginDebug.debug("PluginCookieInfoRequest GOT: " + cookieInfo);
 
-        // Skip the first 5 components. We are guaranteed 5 components, 
-        // so no index -1 to worry about
+        // skip 'plugin' marker
         cookieInfo = cookieInfo.substring(cookieInfo.indexOf(' ') + 1);
+
+        // skip 'PluginCookieInfo' tag
         cookieInfo = cookieInfo.substring(cookieInfo.indexOf(' ') + 1);
+
+        // skip 'reference' tag
         cookieInfo = cookieInfo.substring(cookieInfo.indexOf(' ') + 1);
-        cookieInfo = cookieInfo.substring(cookieInfo.indexOf(' ') + 1);
+
+        // skip reference # and get the rest
         cookieString = cookieInfo.substring(cookieInfo.indexOf(' ') + 1);
 
         setDone(true);