Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Private/caolan/coverity2024 05 12 #9037

Merged
merged 3 commits into from May 13, 2024
Merged

Conversation

caolanm
Copy link
Contributor

@caolanm caolanm commented May 12, 2024

  • Resolves: #
  • Target version: master

Summary

TODO

  • ...

Checklist

  • Code is properly formatted
  • All commits have Change-Id
  • I have run tests with make check
  • I have issued make run and manually verified that everything looks okay
  • Documentation (manuals or wiki) has been updated or is not required

Signed-off-by: Caolán McNamara <[email protected]>
Change-Id: I30949ae9a944a1c254c24a49519b22cf6202a14f
Signed-off-by: Caolán McNamara <[email protected]>
Change-Id: I1fedbde5715f2adb0218bf61ae7014797fb43990
Signed-off-by: Caolán McNamara <[email protected]>
Change-Id: I70e1a5cdec637fc92cab294953599afe667a55cd
@caolanm caolanm requested a review from vmiklos May 12, 2024 15:22
Copy link
Contributor

@vmiklos vmiklos left a comment

Choose a reason for hiding this comment

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

Let's see if this fixes the intermittent unit-save-torture failures I see locally or it's something else. Looks OK anyway, thanks.

@vmiklos vmiklos merged commit 6059896 into master May 13, 2024
14 checks passed
@vmiklos vmiklos deleted the private/caolan/coverity2024-05-12 branch May 13, 2024 06:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

None yet

2 participants