Skip to content

[v16] [buddy] Make sure (*Struct).Fields map is not nil before assignment#53307

Merged
capnspacehook merged 1 commit intobranch/v16from
capnspacehook/backport/v16/53035
Mar 26, 2025
Merged

[v16] [buddy] Make sure (*Struct).Fields map is not nil before assignment#53307
capnspacehook merged 1 commit intobranch/v16from
capnspacehook/backport/v16/53035

Conversation

@capnspacehook
Copy link
Copy Markdown
Contributor

Backport of #53035.

changelog: fix panic when trimming audit log entries

…53035)

* fix: make sure Fields map is not nil

* move new test to existing file, allocate map for struct up front

* make linter happy

---------

Co-authored-by: Erik Jankovic <erik.jankovic@gmail.com>
@capnspacehook capnspacehook added this pull request to the merge queue Mar 23, 2025
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Mar 23, 2025
@capnspacehook capnspacehook added this pull request to the merge queue Mar 26, 2025
Merged via the queue into branch/v16 with commit 4c3e797 Mar 26, 2025
41 checks passed
@capnspacehook capnspacehook deleted the capnspacehook/backport/v16/53035 branch March 26, 2025 01:48
@doggydogworld doggydogworld mentioned this pull request Mar 28, 2025
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.

3 participants