Skip to content

Commit ce51668

Browse files
committed
core: fix NPE in CreateOvaCommand
Prevent failure of ova export when tpm or nvram value are `null`. Signed-off-by: Peter Boden <[email protected]>
1 parent c036868 commit ce51668

File tree

1 file changed

+7
-2
lines changed

1 file changed

+7
-2
lines changed

backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CreateOvaCommand.java

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@
66
import java.util.HashMap;
77
import java.util.List;
88
import java.util.Map;
9+
import java.util.Optional;
910
import java.util.stream.Collectors;
1011

1112
import javax.enterprise.inject.Instance;
@@ -207,8 +208,12 @@ private AnsibleCommandParameters createPackOvaParameters(String ovf,
207208
vars.put("ova_name", getParameters().getName());
208209
vars.put("ovirt_ova_pack_ovf", encodedOvf);
209210
vars.put("ovirt_ova_pack_disks", genDiskParameters(disks, diskIdToPath));
210-
vars.put("ovirt_ova_pack_tpm", tpmData.getValue());
211-
vars.put("ovirt_ova_pack_nvram", nvramData.getValue());
211+
if (!SecretValue.isNull(tpmData) && !tpmData.getValue().isEmpty()) {
212+
vars.put("ovirt_ova_pack_tpm", tpmData.getValue());
213+
}
214+
if (!SecretValue.isNull(nvramData) && !nvramData.getValue().isEmpty()) {
215+
vars.put("ovirt_ova_pack_nvram", nvramData.getValue());
216+
}
212217
vars.put("ovirt_ova_pack_padding", Boolean.toString(compatibilityVersion.greater(Version.v4_6)));
213218
params.setVariables(vars);
214219
return params;

0 commit comments

Comments
 (0)