Federico Simoncelli has uploaded a new change for review.

Change subject: provider: IllegalStateException in getCowVirtualSize
......................................................................

provider: IllegalStateException in getCowVirtualSize

Since JBoss 7.3 (httpclient 4.2) closing an InputStream might also
trigger IllegalStateException. As we were already catching IOException
we should now catch IllegalStateException too.

Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1022569
Change-Id: I20b816c49e60763f4f5b930f309365c0c770e1cd
Signed-off-by: Federico Simoncelli <[email protected]>
---
M 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/provider/OpenStackImageProviderProxy.java
1 file changed, 2 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/03/21003/1

diff --git 
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/provider/OpenStackImageProviderProxy.java
 
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/provider/OpenStackImageProviderProxy.java
index 50987af..5e858bc 100644
--- 
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/provider/OpenStackImageProviderProxy.java
+++ 
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/provider/OpenStackImageProviderProxy.java
@@ -325,8 +325,8 @@
         } finally {
             try {
                 downloadImage.getInputStream().close();
-            } catch (IOException e) {
-                // Silently skip errors on close
+            } catch (IOException | IllegalStateException e) {
+                // Silently skip IOException and IllegalStateException errors
             }
         }
 


-- 
To view, visit http://gerrit.ovirt.org/21003
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I20b816c49e60763f4f5b930f309365c0c770e1cd
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.3
Gerrit-Owner: Federico Simoncelli <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to