You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudstack.apache.org by ro...@apache.org on 2022/04/11 10:37:18 UTC

[cloudstack] branch 4.16 updated: potential null pointer in condition; AYAI9l8k5Irk9_td-cXb (#6237)

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

rohit pushed a commit to branch 4.16
in repository https://gitbox.apache.org/repos/asf/cloudstack.git


The following commit(s) were added to refs/heads/4.16 by this push:
     new ee2ded82001 potential null pointer in condition; AYAI9l8k5Irk9_td-cXb (#6237)
ee2ded82001 is described below

commit ee2ded82001a2a45b945ab9b6a380c50a1f1a437
Author: dahn <da...@shapeblue.com>
AuthorDate: Mon Apr 11 12:37:11 2022 +0200

    potential null pointer in condition; AYAI9l8k5Irk9_td-cXb (#6237)
    
    Co-authored-by: Daan Hoogland <da...@onecht.net>
---
 api/src/main/java/com/cloud/agent/api/storage/OVFParser.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/api/src/main/java/com/cloud/agent/api/storage/OVFParser.java b/api/src/main/java/com/cloud/agent/api/storage/OVFParser.java
index 4c914464211..b4ca8161c5d 100644
--- a/api/src/main/java/com/cloud/agent/api/storage/OVFParser.java
+++ b/api/src/main/java/com/cloud/agent/api/storage/OVFParser.java
@@ -103,7 +103,7 @@ public class OVFParser {
             Node value = childNodes.item(i);
             // Also match if the child's name has a suffix:
             // Example: <rasd:AllocationUnits>
-            if (value != null && (value.getNodeName().equals(childNodeName)) || value.getNodeName().endsWith(":" + childNodeName)) {
+            if (value != null && (value.getNodeName().equals(childNodeName) || value.getNodeName().endsWith(":" + childNodeName))) {
                 return value.getTextContent();
             }
         }