diff --git a/libpod/image/image.go b/libpod/image/image.go index 6a87380c01..33d6a1e546 100644 --- a/libpod/image/image.go +++ b/libpod/image/image.go @@ -1242,6 +1242,14 @@ func areParentAndChild(parent, child *imgspecv1.Image) bool { // the child and candidate parent should share all of the // candidate parent's diff IDs, which together would have // controlled which layers were used + + // Both, child and parent, may be nil when the storage is left in an + // incoherent state. Issue #7444 describes such a case when a build + // has been killed. + if child == nil || parent == nil { + return false + } + if len(parent.RootFS.DiffIDs) > len(child.RootFS.DiffIDs) { return false } diff --git a/libpod/image/layer_tree.go b/libpod/image/layer_tree.go index 3699655fde..18101575e6 100644 --- a/libpod/image/layer_tree.go +++ b/libpod/image/layer_tree.go @@ -32,7 +32,9 @@ func (t *layerTree) toOCI(ctx context.Context, i *Image) (*ociv1.Image, error) { oci, exists := t.ociCache[i.ID()] if !exists { oci, err = i.ociv1Image(ctx) - t.ociCache[i.ID()] = oci + if err == nil { + t.ociCache[i.ID()] = oci + } } return oci, err }