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

fixed all potential crashing when decoding volume ID(CSI identifier) failed #4108

Merged
merged 1 commit into from
Sep 6, 2023

Conversation

crazytaxii
Copy link
Contributor

Describe what this PR does

csi-rbdplugin will crash when decoding volume ID failed due to nil pointer, similar to #4099

Related issues

#4098 #4099 #4107

Signed-off-by: HF <crazytaxii666@gmail.com>
@mergify mergify bot added the bug Something isn't working label Sep 6, 2023
@Madhu-1 Madhu-1 linked an issue Sep 6, 2023 that may be closed by this pull request
@Madhu-1 Madhu-1 requested a review from a team September 6, 2023 11:21
@Madhu-1
Copy link
Collaborator

Madhu-1 commented Sep 6, 2023

@Mergifyio queue

@mergify
Copy link
Contributor

mergify bot commented Sep 6, 2023

queue

✅ The pull request has been merged automatically

The pull request has been merged automatically at 5411a69

@mergify mergify bot added the ok-to-test Label to trigger E2E tests label Sep 6, 2023
@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/k8s-e2e-external-storage/1.27

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/upgrade-tests-cephfs

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/k8s-e2e-external-storage/1.28

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/k8s-e2e-external-storage/1.26

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/upgrade-tests-rbd

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/mini-e2e-helm/k8s-1.27

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/mini-e2e-helm/k8s-1.28

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/mini-e2e-helm/k8s-1.26

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/mini-e2e/k8s-1.27

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/mini-e2e/k8s-1.26

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/mini-e2e/k8s-1.28

@ceph-csi-bot ceph-csi-bot removed the ok-to-test Label to trigger E2E tests label Sep 6, 2023
@mergify mergify bot merged commit 5411a69 into ceph:devel Sep 6, 2023
27 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

csi-rbdplugin crashes when decoding volume ID(CSI identifier) failed
4 participants