From 5cd7a08418ffdfe6978e8f48019eea218e636982 Mon Sep 17 00:00:00 2001 From: Adrian Likins Date: Fri, 16 Dec 2016 11:45:51 -0500 Subject: [PATCH] Check solaris 'smbios' is exists before running it smbios could potentially be None here, possibly causing the solaris 8/9 failures seen at #19282 Fixes #19282 --- lib/ansible/module_utils/facts.py | 2 ++ 1 file changed, 2 insertions(+) diff --git a/lib/ansible/module_utils/facts.py b/lib/ansible/module_utils/facts.py index a99b4eaedb..ed01637b2f 100644 --- a/lib/ansible/module_utils/facts.py +++ b/lib/ansible/module_utils/facts.py @@ -3591,6 +3591,8 @@ class SunOSVirtual(Virtual): else: smbios = self.module.get_bin_path('smbios') + if not smbios: + return rc, out, err = self.module.run_command(smbios) if rc == 0: for line in out.splitlines():