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

pacific: monmaptool: Don't call set_port on an invalid address #40690

Merged
merged 1 commit into from
Apr 26, 2021

Conversation

singuliere
Copy link
Contributor

backport tracker: https://tracker.ceph.com/issues/50131


backport of #38258
parent tracker: https://tracker.ceph.com/issues/48336

this backport was staged using ceph-backport.sh version 16.0.0.6848
find the latest version at https://github.com/ceph/ceph/blob/master/src/script/ceph-backport.sh

@github-actions github-actions bot added the core label Apr 8, 2021
@github-actions github-actions bot added this to the pacific milestone Apr 8, 2021
Verify we parse the entire address argument.

Fixes: https://tracker.ceph.com/issues/48336

Signed-off-by: Brad Hubbard <bhubbard@redhat.com>
Signed-off-by: Kefu Chai <kchai@redhat.com>
(cherry picked from commit c0aa1fd)
@singuliere singuliere requested a review from a team April 14, 2021 12:58
@singuliere singuliere requested review from a team as code owners April 14, 2021 12:58
@singuliere singuliere requested review from aaSharma14 and nizamial09 and removed request for a team April 14, 2021 12:58
@idryomov idryomov changed the base branch from pacific to pacific-saved April 20, 2021 13:18
@idryomov idryomov changed the base branch from pacific-saved to pacific April 20, 2021 13:18
@singuliere
Copy link
Contributor Author

jenkins test this please

@neha-ojha
Copy link
Member

jenkins test api

Copy link
Contributor

@badone badone left a comment

Choose a reason for hiding this comment

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

LGTM

@yuriw yuriw merged commit e08c534 into ceph:pacific Apr 26, 2021
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.

4 participants