Code Review
/
snaps.git
/ commitdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
| commitdiff |
review
|
tree
raw
|
patch
| inline |
side by side
(parent:
0175fee
)
Added additional check for the returned project object is not None.
09/59709/1
author
spisarski
<s.pisarski@cablelabs.com>
Wed, 11 Jul 2018 18:09:51 +0000
(14:09 -0400)
committer
spisarski
<s.pisarski@cablelabs.com>
Wed, 11 Jul 2018 18:09:51 +0000
(14:09 -0400)
There were 3 other instances of this check in neutron_utils.py but
this one was missed and could potentially cause issues.
Change-Id: I599e63e14a9b08ba34f50e060c7dfa5fbdd631f7
Signed-off-by: spisarski <s.pisarski@cablelabs.com>
snaps/openstack/utils/neutron_utils.py
patch
|
blob
|
history
diff --git
a/snaps/openstack/utils/neutron_utils.py
b/snaps/openstack/utils/neutron_utils.py
index
6daa712
..
d82bbf6
100644
(file)
--- a/
snaps/openstack/utils/neutron_utils.py
+++ b/
snaps/openstack/utils/neutron_utils.py
@@
-269,7
+269,7
@@
def get_subnet_by_name(neutron, keystone, subnet_name, project_name=None):
if project_name:
project = keystone_utils.get_project_by_id(
keystone, subnet.project_id)
- if project.name == project_name:
+ if project
and project
.name == project_name:
return subnet
else:
return subnet