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

        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Patrick Corless made changes -
        Field Original Value New Value
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Hide
        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.
        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
        Patrick Corless created issue -

          People

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

            Dates

            • Created:
              Updated:
              Resolved: