Link the two variants of cannotModifyManifest*Reason together.

Signed-off-by: Miloslav Trmač <mitr@redhat.com>
containers · Nov 29, 2021 · a84d4a8 · a84d4a8
1 parent 2aabeb0
commit a84d4a8
Showing 1 changed file with 2 additions and 0 deletions.
diff --git a/copy/copy.go b/copy/copy.go
@@ -433,6 +433,7 @@ func (c *copier) copyMultipleImages(ctx context.Context, policyContext *signatur
 
 	// Determine if we're allowed to modify the manifest list.
 	// If we can, set to the empty string. If we can't, set to the reason why.
+	// Compare, and perhaps keep in sync with, the version in copyOneImage.
 	cannotModifyManifestListReason := ""
 	if len(sigs) > 0 {
 		cannotModifyManifestListReason = "Would invalidate signatures"
@@ -663,6 +664,7 @@ func (c *copier) copyOneImage(ctx context.Context, policyContext *signature.Poli
 
 	// Determine if we're allowed to modify the manifest.
 	// If we can, set to the empty string. If we can't, set to the reason why.
+	// Compare, and perhaps keep in sync with, the version in copyMultipleImages.
 	cannotModifyManifestReason := ""
 	if len(sigs) > 0 {
 		cannotModifyManifestReason = "Would invalidate signatures"