Prevent setting empty container name

Bug #2066939 reported by Nguyen Luu Hoang Minh
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
In Progress
Undecided
Nguyen Luu Hoang Minh

Bug Description

elif backup.container is None:

this condition allows backup container name to be "" (empty string). We should not allow this.

Change to elif not backup.container to avoid empty naming.

Changed in cinder:
assignee: nobody → Nguyen Luu Hoang Minh (minhnlh2)
description: updated
summary: - send error notification on volume detach / attach error
+ Clarify error message on volume attachment
summary: - Clarify error message on volume attachment
+ Enhance variable naming and comment
description: updated
summary: - Enhance variable naming and comment
+ Prevent setting empty container name
description: updated
description: updated
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to cinder (master)

Fix proposed to branch: master
Review: https://review.opendev.org/c/openstack/cinder/+/920321

Changed in cinder:
status: New → In Progress
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.