diff options
author | Colin Walters <walters@verbum.org> | 2022-01-19 19:36:45 -0500 |
---|---|---|
committer | Colin Walters <walters@verbum.org> | 2022-01-20 18:02:58 -0500 |
commit | 2947ded7a8fced28fb3478f556194d12433145ad (patch) | |
tree | a2900ec5e478f15c31ee8270a75249d65b42a00f /pkg/domain/infra/abi/images_test.go | |
parent | 6b59b10248b0bd92127dfc455ca40579d4c396a0 (diff) | |
download | podman-2947ded7a8fced28fb3478f556194d12433145ad.tar.gz podman-2947ded7a8fced28fb3478f556194d12433145ad.tar.bz2 podman-2947ded7a8fced28fb3478f556194d12433145ad.zip |
Don't segfault if an image layer has no creation timestamp
It's optional in the specification, and I initially omitted
it in the ostree code. Now I've fixed the ostree code
to inject a timestamp, but we should clearly avoid segfaulting
on this case.
Signed-off-by: Colin Walters <walters@verbum.org>
Diffstat (limited to 'pkg/domain/infra/abi/images_test.go')
-rw-r--r-- | pkg/domain/infra/abi/images_test.go | 17 |
1 files changed, 17 insertions, 0 deletions
diff --git a/pkg/domain/infra/abi/images_test.go b/pkg/domain/infra/abi/images_test.go index 20ef1b150..e38b9390d 100644 --- a/pkg/domain/infra/abi/images_test.go +++ b/pkg/domain/infra/abi/images_test.go @@ -1,5 +1,22 @@ package abi +import ( + "testing" + + "github.com/containers/common/libimage" + "github.com/stretchr/testify/assert" +) + +// This is really intended to verify what happens with a +// nil pointer in layer.Created, but we'll just sanity +// check round tripping 42. +func TestToDomainHistoryLayer(t *testing.T) { + var layer libimage.ImageHistory + layer.Size = 42 + newLayer := toDomainHistoryLayer(&layer) + assert.Equal(t, layer.Size, newLayer.Size) +} + // // import ( // "context" |