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

schema1: Validate descriptors in AppendReference
pull/2089/head^2
Derek McGowan 2016-11-30 10:26:55 -08:00 committed by GitHub
commit c599955707
1 changed files with 7 additions and 2 deletions

View File

@ -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
}