load_certs: trust dir->Read() slightly less.

scan-build says info->FileName returned from a successful call to
dir->Read() can be NULL.  I don't think that would be a compliant
implementation, but anything's possible.

This patch checks it for NULL-ness before the StrCaseCmp().

Signed-off-by: Peter Jones <pjones@redhat.com>

1 files changed, 2 insertions, 2 deletions

diff --git a/shim.c b/shim.c
index 80b11709..843cb374 100644
--- a/shim.c
+++ b/shim.c
@@ -1496,10 +1496,10 @@ load_certs(EFI_HANDLE image_handle)
 			goto done;
 		}
 
-		if (buffersize == 0)
+		info = (EFI_FILE_INFO *)buffer;
+		if (buffersize == 0 || !info)
 			goto done;
 
-		info = (EFI_FILE_INFO *)buffer;
 		if (StrnCaseCmp(info->FileName, L"shim_certificate", 16) == 0) {
 			load_cert_file(image_handle, info->FileName);
 		}