ICEpush
  1. ICEpush
  2. PUSH-211

NullPointerException in PushContext.backOff

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: None
          • Fix Version/s: EE-3.2.0.GA
          • Component/s: Push Library
          • Labels:
            None
          • Environment:
            ICEmobile, ICEpush

            Description

            The following Exception was observed in place of the expected JSF page view:

            java.lang.NullPointerException
            org.icepush.BlockingConnectionServer.backOff(BlockingConnectionServer.java:91)
            org.icepush.LocalPushGroupManager.backOff(LocalPushGroupManager.java:146)
            org.icepush.PushContext.backOff(PushContext.java:58)
            org.icemobile.impl.application.PushBackOffPhaseListener.beforePhase(PushBackOffPhaseListener.java:45)
            com.sun.faces.lifecycle.Phase.handleBeforePhase(Phase.java:228)
            com.sun.faces.lifecycle.Phase.doPhase(Phase.java:99)
            com.sun.faces.lifecycle.LifecycleImpl.render(LifecycleImpl.java:139)
            javax.faces.webapp.FacesServlet.service(FacesServlet.java:594)

              Activity

              Ascending order - Click to sort in descending order
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #33100 Thu Jan 17 13:54:39 MST 2013 jack.van.ooststroom Fixed JIRA PUSH-211 : NullPointerException in PushContext.backOff
              Files Changed
              Commit graph MODIFY /icepush/trunk/icepush/core/src/main/java/org/icepush/BlockingConnectionServer.java
              Ted Goddard created issue -
              Ted Goddard made changes -
              Field Original Value New Value
              Assignee Jack Van Ooststroom [ jack.van.ooststroom ]
              Ted Goddard made changes -
              Fix Version/s EE-3.2.0.GA [ 10323 ]
              Hide
              Permalink
              Ted Goddard added a comment -

              It appears that the browserID on the BlockingConnection server instance is null (since this is the initial page load, push has not been initialized yet). In this case backOff() should simply return (and log at debug level).

              if (this.browserID.equals(browserID) && delay > 0) {

              Show
              Ted Goddard added a comment - It appears that the browserID on the BlockingConnection server instance is null (since this is the initial page load, push has not been initialized yet). In this case backOff() should simply return (and log at debug level). if (this.browserID.equals(browserID) && delay > 0) {
              Hide
              Permalink
              Jack Van Ooststroom added a comment -

              Sending core/src/main/java/org/icepush/BlockingConnectionServer.java
              Transmitting file data .
              Committed revision 33100.

              Show
              Jack Van Ooststroom added a comment - Sending core/src/main/java/org/icepush/BlockingConnectionServer.java Transmitting file data . Committed revision 33100.
              Hide
              Permalink
              Jack Van Ooststroom added a comment -

              I expanded the if-condition with an additional check != null. Marking this one as FIXED.

              Show
              Jack Van Ooststroom added a comment - I expanded the if-condition with an additional check != null. Marking this one as FIXED.
              Jack Van Ooststroom made changes -
              Status Open [ 1 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Ken Fyten made changes -
              Status Resolved [ 5 ] Closed [ 6 ]

                People

                • Assignee:
                  Jack Van Ooststroom
                  Reporter:
                  Ted Goddard
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  2 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: