dmioem: Move function is_printable to dmidecode.c

Move function is_printable to dmidecode.c so that a single implementation
can be used in both dmidecode.c and dmioem.c.

Author: jdelvare

Diff for: CHANGELOG

@@ -2,6 +2,7 @@
 
 	* dmidecode.c: Fix up invalid DMI type 34 structure length.
 	* dmioem.c: Decode HP-specific DMI types 212 and 219.
+	* dmioem.c: Move function is_printable to dmidecode.c.
 
 2015-05-13  Jean Delvare  <[email protected]>
 

Diff for: dmidecode.c

@@ -81,6 +81,18 @@ static const char *bad_index = "<BAD INDEX>";
  * Type-independant Stuff
  */
 
+/* Returns 1 if the buffer contains only printable ASCII characters */
+int is_printable(const u8 *data, int len)
+{
+	int i;
+
+	for (i = 0; i < len; i++)
+		if (data[i] < 32 || data[i] >= 127)
+			return 0;
+
+	return 1;
+}
+
 const char *dmi_string(const struct dmi_header *dm, u8 s)
 {
 	char *bp = (char *)dm->data;
@@ -2937,18 +2949,14 @@ static void dmi_64bit_memory_error_address(u64 code)
 static void dmi_fixup_type_34(struct dmi_header *h)
 {
 	u8 *p = h->data;
-	int i;
-
- 	if (h->length != 0x10)
-		return;
 
  	/* Make sure the hidden data is ASCII only */
-	for (i = 0x0B; i < 0x10; i++)
-		if (p[i] < 32 || p[i] >= 127)
-			return;
-
-	printf("Invalid entry length (%u). Fixed up to %u.\n", 0x10, 0x0B);
-	h->length = 0x0B;
+	if (h->length == 0x10
+	 && is_printable(p + 0x0B, 0x10 - 0x0B))
+	{
+		printf("Invalid entry length (%u). Fixed up to %u.\n", 0x10, 0x0B);
+		h->length = 0x0B;
+	}
 }
 
 static const char *dmi_management_device_type(u8 code)

Diff for: dmidecode.h

@@ -28,4 +28,5 @@ struct dmi_header
 	u8 *data;
 };
 
+int is_printable(const u8 *data, int len);
 const char *dmi_string(const struct dmi_header *dm, u8 s);

Diff for: dmioem.c

@@ -62,17 +62,6 @@ void dmi_set_vendor(const char *s)
 		dmi_vendor = VENDOR_ACER;
 }
 
-static int is_printable(const u8 *data, int len)
-{
-	int i;
-
-	for (i = 0; i < len; i++)
-		if (data[i] < 32 || data[i] >= 127)
-			return 0;
-
-	return 1;
-}
-
 /*
  * HP-specific data structures are decoded here.
  *