Code Review Request: 6953455 CookieStore.add() cannot handle null URI parameter, contrary to the API specification (original) (raw)

Chris Hegarty chris.hegarty at oracle.com
Wed Jul 4 13:34:04 PDT 2012


Seems like a reasonable candidate for a backport. Are you encountering it in 7?

-Chris

On 4 Jul 2012, at 15:19, Neil Richards <neil.richards at ngmr.net> wrote:

On Fri, 2011-09-30 at 10:08 -0700, Kurchi Hazra wrote:

Hi, The CookieStore.add() method throws a Null Pointer Exception when null is passed as the uri parameter, although this is allowed according to the method spec. The exception is thrown because uri.getHost() is called on a null uri in an effort to add it to the uriIndex, one of the hash maps constituting the CookieStore. The fix would be to simply bypass adding the cookie to the uriIndex when uri is null. The fix involves updates in: src/share/classes/java/net/InMemoryCookieStore.java Webrev : http://cr.openjdk.java.net/~chegar/6953455/webrev.00/webrev/ Thanks, Kurchi Hi, This bug fix looks to have been well bedded into the openjdk 8 code stream at this point. Would this be a good item to be applied to the jdk7u code stream ? Regards, Neil -- Unless stated above: IBM email: neilrichards at uk.ibm.com IBM United Kingdom Limited - Registered in England and Wales with number 741598. Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire PO6 3AU



More information about the jdk7u-dev mailing list