Re: lynx-dev lynx2.8.2dev.3

[brian j. pardy]

address@hidden wrote:
> > Just wanted to verify that one of my other patches regarding this is still 
> > set aside. With only the part that you applied, there'll be a few other 
> > problems with null cookies. 
> I see this (going back to 1 September), grepping for 'pardy':
>       Date: Fri, 6 Nov 1998 18:37:49 -0800
>       From: "brian j. pardy" <address@hidden>
>       To: address@hidden
>       Subject: Re: lynx-dev idea regarding 'Submit' links
> but that doesn't seem applicable.  Perhaps I missed it

Ah, no, not that one.

> > Can resend if you'd like. 
> ok.

Here it is:


diff -cr lynx2-8-1/src/LYCookie.c lynx2.8.2.dev.1.bri/src/LYCookie.c
*** lynx2-8-1/src/LYCookie.c    Fri Nov  6 06:29:41 1998
--- lynx2.8.2.dev.1.bri/src/LYCookie.c  Sun Nov  8 18:59:48 1998
***************
*** 495,500 ****
--- 495,508 ----
        co = NULL;
  
      /*
+      * Don't add the cookie if the value is NULL. - BJP
+      */
+     } else if (co->value[0] == '\0') {
+         CTRACE(tfp, "store_cookie: Value is NULL! Not storing cookie.\n");
+         freeCookie(co);
+         co = NULL;
+ 
+     /*
       *        If it's a replacement for a cookie that had not expired,
       *        and never allow has not been set, add it again without
       *        confirmation. - FM

-- 
Reactor error - core dumped!