Merge pull request #400 from oasisprotocol/kostko/fix/rofl-build-null-eids

kostko · web-flow · commit 4c475f0f3dc6 · 2025-03-17T11:00:32.000+01:00
fix(cmd/rofl): Fix incorrect enclave id list generation
diff --git a/cmd/rofl/build/build.go b/cmd/rofl/build/build.go
@@ -249,7 +249,7 @@ var (
 				fmt.Println("Next time you can also use the --update-manifest flag to apply changes.")
 			case true:
 				// Update the manifest with the given enclave identities, overwriting existing ones.
-				deployment.Policy.Enclaves = make([]sgx.EnclaveIdentity, len(ids))
+				deployment.Policy.Enclaves = make([]sgx.EnclaveIdentity, 0, len(ids))
 				for _, id := range ids {
 					deployment.Policy.Enclaves = append(deployment.Policy.Enclaves, id.Enclave)
 				}

Original file line number	Diff line number	Diff line change
`@@ -249,7 +249,7 @@ var (`
`249`	`249`	`fmt.Println("Next time you can also use the --update-manifest flag to apply changes.")`
`250`	`250`	`case true:`
`251`	`251`	`// Update the manifest with the given enclave identities, overwriting existing ones.`
`252`		`- deployment.Policy.Enclaves = make([]sgx.EnclaveIdentity, len(ids))`
	`252`	`+ deployment.Policy.Enclaves = make([]sgx.EnclaveIdentity, 0, len(ids))`
`253`	`253`	`for _, id := range ids {`
`254`	`254`	`deployment.Policy.Enclaves = append(deployment.Policy.Enclaves, id.Enclave)`
`255`	`255`	`}`