changeset 1481:4a6917683a2d

handle whitespace following cookie attribute value This isn't beautiful, but I don't want to make sweeping changes in cookie parsing if I don't expect to test it much myself. I think leading whitespace would be legal, too, but I won't worry about that for now.
author corvid <corvid@lavabit.com>
date Sun, 27 Dec 2009 23:46:23 +0000
parents 4941b1624ecc
children a749c1b10fbe
files ChangeLog dpi/cookies.c
diffstat 2 files changed, 6 insertions(+), 1 deletions(-) [+]
line wrap: on
line diff
--- a/ChangeLog	Fri Dec 25 07:50:47 2009 +0000
+++ b/ChangeLog	Sun Dec 27 23:46:23 2009 +0000
@@ -56,6 +56,7 @@
  - Enable popup menu below bottom of page content (BUG#856).
  - Handle JPEGs with CMYK color space.
  - Allow keysyms in keysrc.
+ - Improve cookie attribute value parsing (BUG 819)
    Patches: corvid
 +- Support for the letter-spacing property.
    Patch: Johannes Hofmann, corvid
--- a/dpi/cookies.c	Fri Dec 25 07:50:47 2009 +0000
+++ b/dpi/cookies.c	Sun Dec 27 23:46:23 2009 +0000
@@ -769,8 +769,12 @@
       case ' ':
       case '\t':
          if (!broken_syntax && str[0] != '\'' && str[0] != '"') {
+            end = 1;
             *cookie_str = str + i + 1;
-            end = 1;
+            while (**cookie_str == ' ' || **cookie_str == '\t')
+               *cookie_str += 1;
+            if (**cookie_str == ';')
+               *cookie_str += 1;
          }
          break;
       case '\'':