imx9/version/board_mcu_version.c: Check pointer arguments to board_mc…

…u_version It is legal to pass null pointers for those arguments which the caller is not interested of Signed-off-by: Jukka Laitinen <[email protected]>
tiiuae · Sep 1, 2024 · 106932b · 106932b
1 parent 6e6f3b1
commit 106932b
Showing 1 changed file with 11 additions and 3 deletions.
diff --git a/platforms/nuttx/src/px4/nxp/imx9/version/board_mcu_version.c b/platforms/nuttx/src/px4/nxp/imx9/version/board_mcu_version.c
@@ -217,9 +217,17 @@ int board_mcu_version(char *rev, const char **revstr, const char **errata)
 	unsigned len = sizeof(hw_version_table) / sizeof(hw_version_table[0]);
 
 	if (hw_version < len) {
-		*rev = hw_version_table[hw_version].rev;
-		*revstr = hw_version_table[hw_version].revstr;
-		*errata = hw_version_table[hw_version].errata;
+		if (rev) {
+			*rev = hw_version_table[hw_version].rev;
+		}
+
+		if (revstr) {
+			*revstr = hw_version_table[hw_version].revstr;
+		}
+
+		if (errata) {
+			*errata = hw_version_table[hw_version].errata;
+		}
 	}
 
 	return hw_version;