Defining the variable "tmp" before try block

In file "image_utils.py", "tmp" variable initialized in try
block. So, if any exception occurred in the try block, then
"tmp" would not be assigned with any value. Because of this
finally block will raise an exception like "local variable
tmp tried to use before the assignment".

This patch resolves that issue by defining "tmp" variable before
try block.

Change-Id: I9d8c8eaaeba0a7aab7ebfc791b9ddd967f324184
Closes-Bug: #1394548

diff --git a/cinder/image/image_utils.py b/cinder/image/image_utils.py
index 3d280a3ca41c4bdb80bf5f3a17ec2ecfcdc18181..d1dea63a0f9f6e00bceb870f178bc6cdf190331f 100644 (file)
--- a/cinder/image/image_utils.py
+++ b/cinder/image/image_utils.py
@@ -389,11 +389,13 @@ def rename_file(src, dst):
 
 @contextlib.contextmanager
 def temporary_file(*args, **kwargs):
+    tmp = None
     try:
         tmp = create_temporary_file(*args, **kwargs)
         yield tmp
     finally:
-        fileutils.delete_if_exists(tmp)
+        if tmp:
+            fileutils.delete_if_exists(tmp)
 
 
 def temporary_dir():