improved errorlog message by adding null check

Author: saurabhkushwaha438

server/src/main/java/org/apache/cloudstack/vm/ImportVmTasksManagerImpl.java

@@ -161,6 +161,9 @@ public void updateImportVMTaskStep(ImportVmTask importVMTask, DataCenter zone, A
 
     @Override
     public void updateImportVMTaskErrorState(ImportVmTask importVMTask, ImportVmTask.TaskState state, String errorMsg) {
+        if (importVMTask == null) {
+            return;
+        }
         ImportVMTaskVO importVMTaskVO = (ImportVMTaskVO) importVMTask;
         Date updatedDate = DateUtil.now();
         importVMTaskVO.setUpdated(updatedDate);

server/src/main/java/org/apache/cloudstack/vm/UnmanagedVMsManagerImpl.java

@@ -1777,7 +1777,9 @@ protected UserVm importUnmanagedInstanceFromVmwareToKvm(DataCenter zone, Cluster
             return userVm;
         } catch (CloudRuntimeException e) {
             logger.error(String.format("Error importing VM: %s", e.getMessage()), e);
-            importVmTasksManager.updateImportVMTaskErrorState(importVMTask, ImportVmTask.TaskState.Failed, e.getMessage());
+            if (importVMTask != null) {
+                importVmTasksManager.updateImportVMTaskErrorState(importVMTask, ImportVmTask.TaskState.Failed, e.getMessage());
+            }
             ActionEventUtils.onCompletedActionEvent(userId, owner.getId(), EventVO.LEVEL_ERROR, EventTypes.EVENT_VM_IMPORT,
                     cmd.getEventDescription(), null, null, 0);
             throw new ServerApiException(ApiErrorCode.INTERNAL_ERROR, e.getMessage());