From: Sergey Vladimirov <sergey@apache.org>
Date: Sun, 10 Jul 2011 12:59:28 +0000 (+0000)
Subject: NP check
X-Git-Tag: REL_3_8_BETA4~226
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=ce99b827a6c90607ac03304fa16f905f04fada66;p=poi.git

NP check

git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@1144853 13f79535-47bb-0310-9956-ffa450edef68
---

diff --git a/src/scratchpad/src/org/apache/poi/hwpf/model/OldCHPBinTable.java b/src/scratchpad/src/org/apache/poi/hwpf/model/OldCHPBinTable.java
index 6fe191fb84..c1f25dbb37 100644
--- a/src/scratchpad/src/org/apache/poi/hwpf/model/OldCHPBinTable.java
+++ b/src/scratchpad/src/org/apache/poi/hwpf/model/OldCHPBinTable.java
@@ -70,10 +70,11 @@ public final class OldCHPBinTable extends CHPBinTable
         if (chpx != null && tpt.isIndexInTable( chpx.getStartBytes(), chpx.getEndBytes() )) {
             _textRuns.add(chpx);
         } else {
-                    logger.log( POILogger.WARN, "CHPX [",
-                            chpx.getStartBytes(), "; ", chpx.getEndBytes(),
-                            ") (bytes) doesn't have corresponding text pieces "
-                                    + "and will be skipped" );
+                    if ( chpx != null )
+                        logger.log( POILogger.WARN, "CHPX [",
+                                chpx.getStartBytes(), "; ", chpx.getEndBytes(),
+                                ") (bytes) doesn't have corresponding text pieces "
+                                        + "and will be skipped" );
         }
       }
     }
diff --git a/src/scratchpad/src/org/apache/poi/hwpf/model/OldPAPBinTable.java b/src/scratchpad/src/org/apache/poi/hwpf/model/OldPAPBinTable.java
index 0331b997e4..3cc096455e 100644
--- a/src/scratchpad/src/org/apache/poi/hwpf/model/OldPAPBinTable.java
+++ b/src/scratchpad/src/org/apache/poi/hwpf/model/OldPAPBinTable.java
@@ -61,10 +61,11 @@ public final class OldPAPBinTable extends PAPBinTable
         if (papx != null && tpt.isIndexInTable( papx.getStartBytes(), papx.getEndBytes() )) {
             _paragraphs.add(papx);
         } else {
-            logger.log( POILogger.WARN, "PAPX [", papx.getStartBytes(),
-                    "; ", papx.getEndBytes(),
-                    ") (bytes) doesn't have corresponding text pieces "
-                            + "and will be skipped" );
+                    if ( papx != null )
+                        logger.log( POILogger.WARN, "PAPX [",
+                                papx.getStartBytes(), "; ", papx.getEndBytes(),
+                                ") (bytes) doesn't have corresponding text pieces "
+                                        + "and will be skipped" );
         }
       }
     }