ICEpdf
  1. ICEpdf
  2. PDF-1089

xForm transparency buffer out of memory

          Details

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

            Description

            The PDF in question has some very large xform boxes which are content parser tries to render to a buffer for transparency effects. We have code that tries to detect very large buffers but it is failing in this case.

            Once the image bounds are tightened up the PDF renders fine.

              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 6.1.4 [ 13090 ]
              Hide
              Permalink
              Patrick Corless added a comment -

              There were some code inconsistencies for the values that define the maximum buffer size that can be created. The code was setup with a Max_int value for creating the AbstractContentParser which was added for a few corner case encoders and then a second size check was made in the FormDrawCmd command just before the buffer was created.

              The code has been update to use the FormDrawCmd.MAX_IMAGE_SIZE for the FormDrawCmd and AbstractContentParser checks. In general we don't want to go the route of creating an instance of FormDrawCmd if the supporting image is going to be very large. PDF renders fine now and uses significantly less memory to render.

              Show
              Patrick Corless added a comment - There were some code inconsistencies for the values that define the maximum buffer size that can be created. The code was setup with a Max_int value for creating the AbstractContentParser which was added for a few corner case encoders and then a second size check was made in the FormDrawCmd command just before the buffer was created. The code has been update to use the FormDrawCmd.MAX_IMAGE_SIZE for the FormDrawCmd and AbstractContentParser checks. In general we don't want to go the route of creating an instance of FormDrawCmd if the supporting image is going to be very large. PDF renders fine now and uses significantly less memory to render.
              Hide
              Permalink
              Patrick Corless added a comment -

              Marking as fixed.

              Show
              Patrick Corless added a comment - Marking as fixed.
              Patrick Corless made changes -
              Status Open [ 1 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Arran Mccullough made changes -
              Support Case References Support Case #13978 - https://icesoft.my.salesforce.com/5007000001gNsYh
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #49773 Tue Jan 03 09:55:23 MST 2017 patrick.corless PDF-1089 added image size limits before creating a xform buffer.
              Files Changed
              Commit graph MODIFY /icepdf/branches/icepdf-6.1.0/icepdf/core/src/org/icepdf/core/util/content/AbstractContentParser.java
              Commit graph MODIFY /icepdf/branches/icepdf-6.1.0/icepdf/core/src/org/icepdf/core/pobjects/graphics/commands/FormDrawCmd.java
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #49774 Tue Jan 03 09:56:16 MST 2017 patrick.corless PDF-1089 added image size limits before creating a xform buffer.
              Files Changed
              Commit graph MODIFY /icepdf/trunk/icepdf/core/src/org/icepdf/core/util/content/AbstractContentParser.java
              Commit graph MODIFY /icepdf/trunk/icepdf/core/src/org/icepdf/core/pobjects/graphics/commands/FormDrawCmd.java
              Patrick Corless made changes -
              Status Resolved [ 5 ] Closed [ 6 ]

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved: