Skip to content

Fix findbugs RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE in NetworkOrche… - #449

Closed
rsafonseca wants to merge 1 commit into
apache:masterfrom
rsafonseca:findbugs66
Closed

Fix findbugs RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE in NetworkOrche…#449
rsafonseca wants to merge 1 commit into
apache:masterfrom
rsafonseca:findbugs66

Conversation

@rsafonseca

Copy link
Copy Markdown
Contributor

…strator.java

If answer is null, method will throw ConnectionException in line 3016 as satisfied by condition stated in line 3014

…strator.java
If answer is null, method will throw ConnectionException in line 3016 as satisfied by condition stated in line 3014
@asfbot

Copy link
Copy Markdown

cloudstack-pull-requests #429 SUCCESS
This pull request looks good

kishankavala pushed a commit to kishankavala/cloudstack that referenced this pull request Jun 25, 2015
…strator.java If answer is null, method will throw ConnectionException in line 3016 as satisfied by condition stated in line 3014
Signed-off-by: Rohit Yadav <rohit.yadav@shapeblue.com>
This closesapache#449
yadvr pushed a commit that referenced this pull request Jan 20, 2021
Fixes#449
Signed-off-by: Rohit Yadav <rohit.yadav@shapeblue.com>
weizhouapache pushed a commit to weizhouapache/cloudstack that referenced this pull request May 30, 2024
This adds a NPE check on the s_depot.global() which can cause NPE in
case of unit tests, where s_depot is not null but the underlying config
dao is null (not mocked or initialised) via `s_depot.global()` becomes
null.
Signed-off-by: Rohit Yadav <rohit.yadav@shapeblue.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@rsafonseca@asfbot