From a64510815e0857b35c626d4204f1b4c5a18bb525 Mon Sep 17 00:00:00 2001
From: Jerome Lacoste <jerome.lacoste@gmail.com>
Date: Fri, 23 Dec 2011 12:24:39 +0100
Subject: [PATCH 2/7] TIKA-827 try to report something if the exception is not
 properly serializable

---
 .../main/java/org/apache/tika/fork/ForkServer.java |   22 ++++++++++++++++++-
 1 files changed, 20 insertions(+), 2 deletions(-)

diff --git a/tika-core/src/main/java/org/apache/tika/fork/ForkServer.java b/tika-core/src/main/java/org/apache/tika/fork/ForkServer.java
index 96049e9..4d9af19 100644
--- a/tika-core/src/main/java/org/apache/tika/fork/ForkServer.java
+++ b/tika-core/src/main/java/org/apache/tika/fork/ForkServer.java
@@ -19,9 +19,11 @@ package org.apache.tika.fork;
 import java.io.ByteArrayInputStream;
 import java.io.DataInputStream;
 import java.io.DataOutputStream;
+import java.io.File;
 import java.io.IOException;
 import java.io.InputStream;
 import java.io.OutputStream;
+import java.io.PrintWriter;
 import java.lang.reflect.InvocationTargetException;
 import java.lang.reflect.Method;
 import java.net.URL;
@@ -137,8 +139,25 @@ class ForkServer implements Runnable, Checksum {
             output.write(DONE);
         } catch (InvocationTargetException e) {
             output.write(ERROR);
-            ForkObjectInputStream.sendObject(e.getCause(), output);
+            // warning if an exception isn't really serializable, this can fail.
+            try {
+                ForkObjectInputStream.sendObject(e.getCause(), output);
+            } catch (IOException e1) {
+                File tempFile = dumpStackTraceIntoTempFile(e);
+                ForkObjectInputStream.sendObject("Unable to properly report originating error. Strack trace dumped: " + tempFile.getAbsolutePath(), output);
+            }
+        }
+    }
+
+    private File dumpStackTraceIntoTempFile(InvocationTargetException e) throws IOException {
+        File tempFile = File.createTempFile("tika", ".trace");
+        PrintWriter pw = new PrintWriter(tempFile);
+        try {
+            e.printStackTrace(pw);
+        } finally {
+            pw.close();
         }
+        return tempFile;
     }
 
     private Method getMethod(Object object, String name) {
@@ -161,7 +180,6 @@ class ForkServer implements Runnable, Checksum {
      * is expected to be preceded by a size integer, that is used for reading
      * the entire serialization into a memory before deserializing it.
      *
-     * @param input input stream from which the serialized object is read
      * @param loader class loader to be used for loading referenced classes
      * @throws IOException if the object could not be deserialized
      * @throws ClassNotFoundException if a referenced class is not found
-- 
1.7.7