You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by si...@apache.org on 2019/10/16 10:48:55 UTC

[camel] branch master updated: CAMEL-14072 FileInputStreamCache does not delete file if file system is full

This is an automated email from the ASF dual-hosted git repository.

siano pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/camel.git


The following commit(s) were added to refs/heads/master by this push:
     new 9dfd6a5  CAMEL-14072 FileInputStreamCache does not delete file if file system is full
9dfd6a5 is described below

commit 9dfd6a5559851b08584d3eeae8cb9ee0ba00d760
Author: Stephan Siano <st...@sap.com>
AuthorDate: Wed Oct 16 08:28:32 2019 +0200

    CAMEL-14072 FileInputStreamCache does not delete file if file system is full
    
    Thanks to André Weickel for reporting and proposed fix
---
 .../apache/camel/converter/stream/FileInputStreamCache.java    | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/core/camel-support/src/main/java/org/apache/camel/converter/stream/FileInputStreamCache.java b/core/camel-support/src/main/java/org/apache/camel/converter/stream/FileInputStreamCache.java
index c880aaf..99b46db 100644
--- a/core/camel-support/src/main/java/org/apache/camel/converter/stream/FileInputStreamCache.java
+++ b/core/camel-support/src/main/java/org/apache/camel/converter/stream/FileInputStreamCache.java
@@ -206,14 +206,14 @@ public final class FileInputStreamCache extends InputStream implements StreamCac
                                 if (outputStream != null) {
                                     outputStream.close();
                                 }
-                                try {
-                                    cleanUpTempFile();
-                                } catch (Exception e) {
-                                    LOG.warn("Error deleting temporary cache file: " + tempFile + ". This exception will be ignored.", e);
-                                }
                             } catch (Exception e) {
                                 LOG.warn("Error closing streams. This exception will be ignored.", e);
                             }
+                            try {
+                                cleanUpTempFile();
+                            } catch (Exception e) {
+                                LOG.warn("Error deleting temporary cache file: " + tempFile + ". This exception will be ignored.", e);
+                            }
                         }
                     }