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

[rohit]

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 <daan.hoogl...@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 <d...@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();
             }
         }