Skip to content

[26.1 backport] builder/mobyexporter: Add missing nil check#47986

Merged
thaJeztah merged 1 commit intomoby:26.1from
vvoland:v26.1-47985
Jun 14, 2024
Merged

[26.1 backport] builder/mobyexporter: Add missing nil check#47986
thaJeztah merged 1 commit intomoby:26.1from
vvoland:v26.1-47985

Conversation

@vvoland
Copy link
Copy Markdown
Contributor

@vvoland vvoland commented Jun 14, 2024

Add a nil check to handle a case where the image config JSON would deserialize into a nil map.

- What I did

- How I did it

- How to verify it
TestPatchImageConfig

- A picture of a cute animal (not mandatory but encouraged)

Add a nil check to handle a case where the image config JSON would
deserialize into a nil map.

Signed-off-by: Paweł Gronowski <[email protected]>
(cherry picked from commit 642242a)
Signed-off-by: Paweł Gronowski <[email protected]>
@vvoland vvoland added this to the 26.1.5 milestone Jun 14, 2024
@vvoland vvoland self-assigned this Jun 14, 2024
@vvoland vvoland requested a review from tonistiigi as a code owner June 14, 2024 14:18
Copy link
Copy Markdown
Member

@thaJeztah thaJeztah left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@thaJeztah thaJeztah merged commit f533464 into moby:26.1 Jun 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants