ICEpdf
  1. ICEpdf
  2. PDF-851

RC4 AES encrypted document is prompting for password

          Details

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

            Description

            The file in question uses 128 bit RC4 AES encryption. For some reason the password callback is being fired by the security handler. Further investigation is needed.

              Activity

              Ascending order - Click to sort in descending order
              Patrick Corless created issue -
              Patrick Corless made changes -
              Field Original Value New Value
              Fix Version/s 5.1.2 [ 11872 ]
              Arran Mccullough made changes -
              Support Case References Support Case #13253 - https://icesoft.my.salesforce.com/5007000000zOLZG
              Hide
              Permalink
              Patrick Corless added a comment -

              After quite a bit of tracing it appears that we had an issue with how we are calculating the encryption key for v4 of the specification. The main issue was with regards to step 6 implementation or lack there of.

              "If document metadata is not being encrypted, pass 4 bytes with the value of 0xFFFFFFFF to the MD5 hash, Security handlers of revision 4 or greater)"

              Interestingly enough this bug has been around for quite some time.

              Show
              Patrick Corless added a comment - After quite a bit of tracing it appears that we had an issue with how we are calculating the encryption key for v4 of the specification. The main issue was with regards to step 6 implementation or lack there of. "If document metadata is not being encrypted, pass 4 bytes with the value of 0xFFFFFFFF to the MD5 hash, Security handlers of revision 4 or greater)" Interestingly enough this bug has been around for quite some time.
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #43922 Mon Jan 19 15:24:18 MST 2015 patrick.corless PDF-851 fixed an issue with the encryption key algorithm with regards to step 6 for documents that don't have encrypted metadata
              Files Changed
              Commit graph MODIFY /icepdf/trunk/icepdf/core/src/org/icepdf/core/pobjects/security/StandardEncryption.java
              Commit graph MODIFY /icepdf/trunk/icepdf/core/src/org/icepdf/core/pobjects/security/EncryptionDictionary.java
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #43923 Mon Jan 19 15:24:42 MST 2015 patrick.corless PDF-851 fixed an issue with the encryption key algorithm with regards to step 6 for documents that don't have encrypted metadata.
              Files Changed
              Commit graph MODIFY /icepdf/branches/icepdf-5.0.1/icepdf/core/src/org/icepdf/core/pobjects/security/StandardEncryption.java
              Commit graph MODIFY /icepdf/branches/icepdf-5.0.1/icepdf/core/src/org/icepdf/core/pobjects/security/EncryptionDictionary.java
              Hide
              Permalink
              Patrick Corless added a comment -

              Marking as resolved.

              Show
              Patrick Corless added a comment - Marking as resolved.
              Patrick Corless made changes -
              Status Open [ 1 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #43925 Tue Jan 20 08:45:11 MST 2015 patrick.corless PDF-851 fixed an issue with the encryption key algorithm with regards to step 6 for documents that don't have encrypted metadata.
              Files Changed
              Commit graph MODIFY /icepdf/branches/icepdf-5.0.1/icepdf/core/src/org/icepdf/core/pobjects/security/StandardEncryption.java
              Patrick Corless made changes -
              Status Resolved [ 5 ] Closed [ 6 ]

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved: