You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@poi.apache.org by ta...@apache.org on 2013/09/27 21:35:15 UTC

svn commit: r1527041 - /poi/trunk/src/scratchpad/src/org/apache/poi/hslf/extractor/PowerPointExtractor.java

Author: tallison
Date: Fri Sep 27 19:35:14 2013
New Revision: 1527041

URL: http://svn.apache.org/r1527041
Log:
Potential NPE in PPExtractor; found while working on TIKA-1171

Modified:
    poi/trunk/src/scratchpad/src/org/apache/poi/hslf/extractor/PowerPointExtractor.java

Modified: poi/trunk/src/scratchpad/src/org/apache/poi/hslf/extractor/PowerPointExtractor.java
URL: http://svn.apache.org/viewvc/poi/trunk/src/scratchpad/src/org/apache/poi/hslf/extractor/PowerPointExtractor.java?rev=1527041&r1=1527040&r2=1527041&view=diff
==============================================================================
--- poi/trunk/src/scratchpad/src/org/apache/poi/hslf/extractor/PowerPointExtractor.java (original)
+++ poi/trunk/src/scratchpad/src/org/apache/poi/hslf/extractor/PowerPointExtractor.java Fri Sep 27 19:35:14 2013
@@ -231,9 +231,11 @@ public final class PowerPointExtractor e
                             }
                             TextShape tsh = (TextShape)sh;
                             String text = tsh.getText();
-                            ret.append(text);
-                            if (!text.endsWith("\n")) {
-                                ret.append("\n");
+                            if (text != null){
+                                ret.append(text);
+                                if (!text.endsWith("\n")) {
+                                    ret.append("\n");
+                                }
                             }
                         }
                     }



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@poi.apache.org
For additional commands, e-mail: commits-help@poi.apache.org