libstb: remove subid argument from sb_verify and tb_measure

We always verify and measure an image as a whole, never its subpartition
(if exists).

This removes the subid argument from sb_verify() and tb_measure()
functions, and also reflects the change to the callers, STB interface
and STB documentation.

Signed-off-by: Claudio Carvalho <cclaudio@linux.vnet.ibm.com>
Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>

2 files changed, 14 insertions, 17 deletions

diff --git a/libstb/stb.c b/libstb/stb.c
index 4dc28a9..be68a50 100644
--- a/libstb/stb.c
+++ b/libstb/stb.c
@@ -202,7 +202,7 @@ int stb_final(void)
 	return rc;
 }
 
-int tb_measure(enum resource_id id, uint32_t subid, void *buf, size_t len)
+int tb_measure(enum resource_id id, void *buf, size_t len)
 {
 	int rc, r;
 	uint8_t digest[SHA512_DIGEST_LENGTH];
@@ -234,8 +234,8 @@ int tb_measure(enum resource_id id, uint32_t subid, void *buf, size_t len)
 		 * come from the resource load framework and likely indicates a
 		 * bug in the framework.
 		 */
-		prlog(PR_ERR, "STB: %s failed: resource %s%d, buf null\n",
-		      __func__, resource_map[r].name, subid);
+		prlog(PR_ERR, "STB: %s failed: resource %s, buf null\n",
+		      __func__, resource_map[r].name);
 		return STB_ARG_ERROR;
 	}
 	memset(digest, 0, SHA512_DIGEST_LENGTH);
@@ -290,12 +290,12 @@ int tb_measure(enum resource_id id, uint32_t subid, void *buf, size_t len)
 			 EV_ACTION, resource_map[r].name);
 	if (rc)
 		return rc;
-	prlog(PR_NOTICE, "STB: %s%d measured to pcr%d\n", resource_map[r].name,
-	      subid, resource_map[r].pcr);
+	prlog(PR_NOTICE, "STB: %s measured to pcr%d\n", resource_map[r].name,
+	      resource_map[r].pcr);
 	return 0;
 }
 
-int sb_verify(enum resource_id id, uint32_t subid, void *buf, size_t len)
+int sb_verify(enum resource_id id, void *buf, size_t len)
 {
 	int r;
 	const char *name = NULL;
@@ -315,7 +315,7 @@ int sb_verify(enum resource_id id, uint32_t subid, void *buf, size_t len)
 		 * measured if trusted mode is on.
 		 */
 		prlog(PR_WARNING, "STB: verifying the non-expected "
-		      "resource %d/%d\n", id, subid);
+		      "resource %d\n", id);
 	else
 		name = resource_map[r].name;
 	if (!rom_driver || !rom_driver->verify) {
@@ -323,16 +323,16 @@ int sb_verify(enum resource_id id, uint32_t subid, void *buf, size_t len)
 		sb_enforce();
 	}
 	if (!buf || len < SECURE_BOOT_HEADERS_SIZE) {
-		prlog(PR_EMERG, "STB: %s arg error: id %d/%d, buf %p, len %zd\n",
-		      __func__, id, subid, buf, len);
+		prlog(PR_EMERG, "STB: %s arg error: id %d, buf %p, len %zd\n",
+		      __func__, id, buf, len);
 		sb_enforce();
 	}
 	if (rom_driver->verify(buf)) {
-		prlog(PR_EMERG, "STB: %s failed: resource %s%d, "
-		      "eyecatcher 0x%016llx\n", __func__, name, subid,
+		prlog(PR_EMERG, "STB: %s failed: resource %s, "
+		      "eyecatcher 0x%016llx\n", __func__, name,
 		      *((uint64_t*)buf));
 		sb_enforce();
 	}
-	prlog(PR_NOTICE, "STB: %s%d verified\n", name, subid);
+	prlog(PR_NOTICE, "STB: %s verified\n", name);
 	return 0;
 }
diff --git a/libstb/stb.h b/libstb/stb.h
index 2141cf1..6ca44ea 100644
--- a/libstb/stb.h
+++ b/libstb/stb.h
@@ -39,7 +39,6 @@ extern int stb_final(void);
 /**
  * sb_verify - verify a resource
  * @id   : resource id
- * @subid: subpartition id
  * @buf  : data to be verified
  * @len  : buf length
  *
@@ -51,13 +50,12 @@ extern int stb_final(void);
  *
  * returns: 0 otherwise the boot process is aborted
  */
-extern int sb_verify(enum resource_id id, uint32_t subid, void *buf, size_t len);
+extern int sb_verify(enum resource_id id, void *buf, size_t len);
 
 
 /**
  * tb_measure - measure a resource
  * @id    : resource id
- * @subid : subpartition id
  * @buf   : data to be measured
  * @len   : buf length
  *
@@ -69,7 +67,6 @@ extern int sb_verify(enum resource_id id, uint32_t subid, void *buf, size_t len)
  *
  * returns: 0 or an error as defined in status_codes.h
  */
-extern int tb_measure(enum resource_id id, uint32_t subid, void *buf,
-		      size_t len);
+extern int tb_measure(enum resource_id id, void *buf, size_t len);
 
 #endif /* __STB_H */