Merge pull request #2087 from aaronlehmann/missing-digest-panic

schema1: Validate descriptors in AppendReference
distribution · Nov 30, 2016 · c599955 · c599955
2 parents bf27f26 + ec5fa1f
commit c599955
Showing 1 changed file with 7 additions and 2 deletions.
diff --git a/manifest/schema1/config_builder.go b/manifest/schema1/config_builder.go
@@ -240,8 +240,13 @@ func (mb *configManifestBuilder) emptyTar(ctx context.Context) (digest.Digest, e
 
 // AppendReference adds a reference to the current ManifestBuilder
 func (mb *configManifestBuilder) AppendReference(d distribution.Describable) error {
-	// todo: verification here?
-	mb.descriptors = append(mb.descriptors, d.Descriptor())
+	descriptor := d.Descriptor()
+
+	if err := descriptor.Digest.Validate(); err != nil {
+		return err
+	}
+
+	mb.descriptors = append(mb.descriptors, descriptor)
 	return nil
 }