This project is archived and is in readonly mode.

#87 ✓resolved
Alcmene

get() applied to an empty cookie through a Hash.cookie instance creates a stack overflow

Reported by Alcmene | June 3rd, 2008 @ 11:25 AM | in 1.2.0

The get() method applied to an empty cookie through a Hash.Cookie instance creates a stack overflow. A workaround is to use hash.get() instead of get(). This is necessary only for an empty cookie.

I already filed this at dev.mootools before, I was requested for a demo page, here it is : defect demo

It is not very important, but very hard to debug !

The version of MooTools used is the latest available through the downloads page. I tried to use the latest Hash.Cookie from trunk, but it didn't change anything (actually, the last modification date of Hash.Cookie makes it clear…).

Bug confirmed on Safari 3.1, WebKit r34257, WebKit r34318 (with SquirrelFish, Firefox 2.0.0.14, Firefox 3b5 and Opera 9 (all on a Mac).

Comments and changes to this ticket

  • Alcmene

    Alcmene June 3rd, 2008 @ 11:28 AM

    Sorry for the previous ticket being empty, it seems like LightHouse doesn't like text pasting…

    Also, the milestone should be 1.2, but I don't see where I can edit it !

    Well, that's still better than a 502… ;)

  • Thomas Aylott

    Thomas Aylott June 3rd, 2008 @ 03:20 PM

    • Milestone changed from 2.0 to 1.2.0
  • Jan Kassens

    Jan Kassens June 5th, 2008 @ 09:07 PM

    • State changed from “new” to “open”
    • Assigned user changed from “Valerio” to “Jan Kassens”

    in Hash.Cookie.save(), this.erase(); needs to be replaced with this.dispose(), i'm going to commit this, as soon as the revisioning system is ready.

  • Jan Kassens

    Jan Kassens June 13th, 2008 @ 11:53 AM

    • State changed from “open” to “resolved”

    was already fixed

Create your profile

Help contribute to this project by taking a few moments to create your personal profile. Create your profile »

Shared Ticket Bins

Pages