glassfish
  1. glassfish
  2. GLASSFISH-2421

HttpSession.removeAttribute(null) should throw IllegalArgumentException instead of NullPointerException

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 9.1pe
    • Fix Version/s: 9.1pe
    • Component/s: web_container
    • Labels:
      None
    • Environment:

      Operating System: All
      Platform: Sun

    • Issuezilla Id:
      2,421
    • Status Whiteboard:
      Hide

      fixed-pwc12

      Show
      fixed-pwc12

      Description

      HttpSession.removeAttribute(null) should throw IllegalArgumentException instead
      of NullPointerException

        Activity

        Hide
        kmeduri added a comment -

        Fixed in PWC12Dev_Branch:

        Checking in LocalStrings.properties;
        /cvs/glassfish/appserv-webtier/src/java/org/apache/catalina/session/LocalStrings.properties,v
        <-- LocalStrings.properties
        new revision: 1.2.10.1; previous revision: 1.2
        done
        Checking in StandardSession.java;
        /cvs/glassfish/appserv-webtier/src/java/org/apache/catalina/session/StandardSession.java,v
        <-- StandardSession.java
        new revision: 1.18.6.7; previous revision: 1.18.6.6
        done

        Show
        kmeduri added a comment - Fixed in PWC12Dev_Branch: Checking in LocalStrings.properties; /cvs/glassfish/appserv-webtier/src/java/org/apache/catalina/session/LocalStrings.properties,v <-- LocalStrings.properties new revision: 1.2.10.1; previous revision: 1.2 done Checking in StandardSession.java; /cvs/glassfish/appserv-webtier/src/java/org/apache/catalina/session/StandardSession.java,v <-- StandardSession.java new revision: 1.18.6.7; previous revision: 1.18.6.6 done
        Hide
        kmeduri added a comment -

        Fixed in PWC12Dev_Branch:
        -------------------------
        Checking in LocalStrings.properties;
        /cvs/glassfish/appserv-webtier/src/java/org/apache/catalina/session/LocalStrings.properties,v
        <-- LocalStrings.properties
        new revision: 1.2.10.1; previous revision: 1.2
        done
        Checking in StandardSession.java;
        /cvs/glassfish/appserv-webtier/src/java/org/apache/catalina/session/StandardSession.java,v
        <-- StandardSession.java
        new revision: 1.18.6.7; previous revision: 1.18.6.6
        done

        Show
        kmeduri added a comment - Fixed in PWC12Dev_Branch: ------------------------- Checking in LocalStrings.properties; /cvs/glassfish/appserv-webtier/src/java/org/apache/catalina/session/LocalStrings.properties,v <-- LocalStrings.properties new revision: 1.2.10.1; previous revision: 1.2 done Checking in StandardSession.java; /cvs/glassfish/appserv-webtier/src/java/org/apache/catalina/session/StandardSession.java,v <-- StandardSession.java new revision: 1.18.6.7; previous revision: 1.18.6.6 done
        Hide
        gfbugbridge added a comment -

        <BT6525453>

        Show
        gfbugbridge added a comment - <BT6525453>
        Hide
        jluehe added a comment -

        Ported to trunk:

        Checking in LocalStrings.properties;
        /cvs/glassfish/appserv-webtier/src/java/org/apache/catalina/session/LocalStrings.properties,v
        <-- LocalStrings.properties
        new revision: 1.3; previous revision: 1.2
        done
        Checking in StandardSession.java;
        /cvs/glassfish/appserv-webtier/src/java/org/apache/catalina/session/StandardSession.java,v
        <-- StandardSession.java
        new revision: 1.31; previous revision: 1.30
        done

        Show
        jluehe added a comment - Ported to trunk: Checking in LocalStrings.properties; /cvs/glassfish/appserv-webtier/src/java/org/apache/catalina/session/LocalStrings.properties,v <-- LocalStrings.properties new revision: 1.3; previous revision: 1.2 done Checking in StandardSession.java; /cvs/glassfish/appserv-webtier/src/java/org/apache/catalina/session/StandardSession.java,v <-- StandardSession.java new revision: 1.31; previous revision: 1.30 done
        Hide
        kmeduri added a comment -

        Synch'ed up PWC12Dev_Branch with trunk for a minor change in log message:

        Checking in LocalStrings.properties;
        /cvs/glassfish/appserv-webtier/src/java/org/apache/catalina/session/LocalStrings.properties,v
        <-- LocalStrings.properties
        new revision: 1.2.10.2; previous revision: 1.2.10.1
        done

        Show
        kmeduri added a comment - Synch'ed up PWC12Dev_Branch with trunk for a minor change in log message: Checking in LocalStrings.properties; /cvs/glassfish/appserv-webtier/src/java/org/apache/catalina/session/LocalStrings.properties,v <-- LocalStrings.properties new revision: 1.2.10.2; previous revision: 1.2.10.1 done

          People

          • Assignee:
            kmeduri
            Reporter:
            kmeduri
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: