ICEpdf
  1. ICEpdf
  2. PDF-361

Defs.property() security exception results in infinit loop

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: 4.2.2
          • Fix Version/s: 4.3
          • Component/s: Core/Parsing
          • Labels:
            None
          • Environment:
            any

            Description

            The method public static String property(String name, String defaultValue) catches a security exception and then recalls the same method. Doesn't make much sense but however I suspect it was related to a re-factoring error in the class with some fall back code was removed that stored system properties in a secondary map.... long story there.

              Activity

              Ascending order - Click to sort in descending order
              Patrick Corless created issue -
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #27172 Thu Jan 12 13:27:41 MST 2012 patrick.corless PDF-361 removed recursive calls to getProperty and setProperty when a security exception occurs. The odd code was likely the result of the old Defs class that would hold a cache of system properties.
              Files Changed
              Commit graph MODIFY /icepdf/trunk/icepdf/core/src/org/icepdf/core/util/Defs.java
              Hide
              Permalink
              Patrick Corless added a comment -

              Update the Defs class to simply log the security exception. Pretty sure this was the side effect of an old properties cache being removing incorrectly. Marking as resolved.

              Show
              Patrick Corless added a comment - Update the Defs class to simply log the security exception. Pretty sure this was the side effect of an old properties cache being removing incorrectly. Marking as resolved.
              Patrick Corless made changes -
              Field Original Value New Value
              Status Open [ 1 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Ken Fyten made changes -
              Status Resolved [ 5 ] Closed [ 6 ]

                People

                • Assignee:
                  Patrick Corless
                  Reporter:
                  Patrick Corless
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  0 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: