Skip to content

fix nullptr dereference in prune_requests_older_than (backport #2008)#2065

Merged
ivanpauno merged 1 commit intohumblefrom
mergify/bp/humble/pr-2008
Dec 13, 2022
Merged

fix nullptr dereference in prune_requests_older_than (backport #2008)#2065
ivanpauno merged 1 commit intohumblefrom
mergify/bp/humble/pr-2008

Conversation

@mergify
Copy link
Copy Markdown
Contributor

@mergify mergify bot commented Dec 13, 2022

This is an automatic backport of pull request #2008 done by Mergify.


Mergify commands and options

More conditions and actions can be found in the documentation.

You can also trigger Mergify actions by commenting on this pull request:

  • @Mergifyio refresh will re-evaluate the rules
  • @Mergifyio rebase will rebase this PR on its base branch
  • @Mergifyio update will merge the base branch into this PR
  • @Mergifyio backport <destination> will backport this PR on <destination> branch

Additionally, on Mergify dashboard you can:

  • look at your merge queues
  • generate the Mergify configuration with the config editor.

Finally, you can contact us on https://mergify.com

* fix nullptr dereference in prune_requests_older_than

Signed-off-by: akela1101 <akela1101@gmail.com>

* add tests for prune_requests_older_than

Signed-off-by: akela1101 <akela1101@gmail.com>

* Update rclcpp/test/rclcpp/test_client.cpp

Co-authored-by: Chen Lihui <lihui.chen@sony.com>
Signed-off-by: akela1101 <akela1101@gmail.com>

Signed-off-by: akela1101 <akela1101@gmail.com>
Co-authored-by: Chen Lihui <lihui.chen@sony.com>
(cherry picked from commit 1ac37b6)
@ivanpauno
Copy link
Copy Markdown
Member

  • Linux Build Status
  • Linux-aarch64 Build Status
  • Windows Build Status

@ivanpauno ivanpauno merged commit f9050cd into humble Dec 13, 2022
@delete-merged-branch delete-merged-branch bot deleted the mergify/bp/humble/pr-2008 branch December 13, 2022 18:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

No open projects

Development

Successfully merging this pull request may close these issues.

3 participants