commit 6c87d542920c623e25424cecbd0e5fb1f7a6c1b2
author Stefan Berger <stefanb@linux.ibm.com> Wed May 25 15:55:27 2022 -0400
committer Stefan Berger <stefanb@us.ibm.com> Wed May 25 18:54:58 2022 -0400
tree a307f92ec056f8c244f9844ce41b759bd451aa8b
parent 760ec8ea96059a65e047102a406dca6b5a236d27

swtpm: Replace malloc + strcpy with strdup and handle OOM case

Signed-off-by: Stefan Berger <stefanb@linux.ibm.com>

src/swtpm/swtpm_nvstore_linear.c

diff --git a/src/swtpm/swtpm_nvstore_linear.c b/src/swtpm/swtpm_nvstore_linear.c
index eda7793..6e4ad6f 100644
--- a/src/swtpm/swtpm_nvstore_linear.c
+++ b/src/swtpm/swtpm_nvstore_linear.c
@@ -241,8 +241,12 @@
         return TPM_FAIL;
     }
 
-    state.loaded_uri = malloc(strlen(uri) + 1);
-    strcpy(state.loaded_uri, uri);
+    state.loaded_uri = strdup(uri);
+    if (!state.loaded_uri) {
+        logprintf(STDERR_FILENO,
+                  "SWTPM_NVRAM_PrepareLinear: Out of memory\n");
+        return TPM_FAIL;
+    }
 
     /* TODO: Parse URI prefixes ("iscsi://", "rbd://", etc...) */
     state.ops = &nvram_linear_file_ops;