ICEfaces
  1. ICEfaces
  2. ICE-9939

NPE in CompatResourceServlet on URL's without path info ("http://host/url-pattern/")

          Details

          • Type: Bug Bug
          • Status: Open
          • Priority: Major Major
          • Resolution: Unresolved
          • Affects Version/s: 3.3
          • Fix Version/s: None
          • Component/s: Framework
          • Labels:
            None
          • Environment:
            Windows 7 Pro, Chrome 33.
          • Workaround Exists:
            Yes
          • Workaround Description:
            Hide
            swap path variable and checking string:
                if (("..").equals(path)) { ....

            as i see it will not affect any code after, it ready to treat NULL value.
            Show
            swap path variable and checking string:     if (("..").equals(path)) { .... as i see it will not affect any code after, it ready to treat NULL value.

            Description

            CompatResourceServlet throw Null pointer exception on url's without path info:
            http://host/servlet-url-pattern
            http://host/servlet-url-pattern/

            result of httpServletRequest.getPathInfo() for above url's is NULL

            Place where exception is thrown:

            77: String path = httpServletRequest.getPathInfo();
            81: if(path.contains("..")) { ......

              Issue Links

              duplicates

              Bug - A problem which impairs or prevents the functions of the product. ICE-9942 NPE when pathInfo is not available

              • Major - Major loss of function.
              • Closed

                Activity

                Hide
                Permalink
                Kairat Rakhimov added a comment -

                Issue is already fixed by Deryk Sinotte

                Show
                Kairat Rakhimov added a comment - Issue is already fixed by Deryk Sinotte

                  People

                  • Assignee:
                    Unassigned
                    Reporter:
                    Kairat Rakhimov
                  • Votes:
                    0 Vote for this issue
                    Watchers:
                    1 Start watching this issue

                    Dates

                    • Created:
                      Updated: