https://bz.apache.org/bugzilla/show_bug.cgi?id=61584

            Bug ID: 61584
           Summary: NPE in XDGFPage getPageOffset: check for x instead of
                    y
           Product: POI
           Version: unspecified
          Hardware: All
                OS: All
            Status: NEW
          Severity: normal
          Priority: P2
         Component: XDGF
          Assignee: [email protected]
          Reporter: [email protected]
  Target Milestone: ---

XDGFPage getPageOffset contains a wrong check for null, causing an NPE for
.vsdx documents that don't have the YRulerOrigin set.

The code in question has been there ever since this was contributed, so it
appears to affect all versions.

,-- XDGFPage.java, lines 99ff [1] --
   99    if (xoffcell != null)
  100        xoffset = Double.parseDouble(xoffcell.getValue());
  101
  102    if (xoffcell != null)
  103        yoffset = Double.parseDouble(yoffcell.getValue());
`-----

The fix is trivial, line 102 should check yoffcell for being null.

[1]
https://apache.googlesource.com/poi/+/refs/heads/trunk/src/ooxml/java/org/apache/poi/xdgf/usermodel/XDGFPage.java#102

-- 
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to