aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
-rw-r--r--ChangeLog8
-rw-r--r--NEWS1
-rw-r--r--plugin/icedteanp/java/sun/applet/PluginCookieInfoRequest.java10
3 files changed, 16 insertions, 3 deletions
diff --git a/ChangeLog b/ChangeLog
index ca9f276..5fb5daf 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,11 @@
+2010-12-22 Deepak Bhole <[email protected]>
+
+ 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 <[email protected]>
* netx/net/sourceforge/jnlp/controlpanel/AdvancedProxySettingsPane.java,
diff --git a/NEWS b/NEWS
index ec74468..787561f 100644
--- a/NEWS
+++ b/NEWS
@@ -23,6 +23,7 @@ New in release 1.0 (2010-XX-XX):
- PR593: Increment of invalidated iterator in IcedTeaPluginUtils (patch from [email protected])
- 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 --git a/plugin/icedteanp/java/sun/applet/PluginCookieInfoRequest.java b/plugin/icedteanp/java/sun/applet/PluginCookieInfoRequest.java
index 89ef98d..7671c40 100644
--- a/plugin/icedteanp/java/sun/applet/PluginCookieInfoRequest.java
+++ b/plugin/icedteanp/java/sun/applet/PluginCookieInfoRequest.java
@@ -56,12 +56,16 @@ public class PluginCookieInfoRequest extends PluginCallRequest {
PluginDebug.debug("PluginCookieInfoRequest GOT: " + cookieInfo);
- // Skip the first 5 components. We are guaranteed 5 components,
- // so no index -1 to worry about
- cookieInfo = cookieInfo.substring(cookieInfo.indexOf(' ') + 1);
+ // 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);
+
+ // skip reference # and get the rest
cookieString = cookieInfo.substring(cookieInfo.indexOf(' ') + 1);
setDone(true);