FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Fix findbugs RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE in NetworkOrche… by rsafonseca · Pull Request #449 · apache/cloudstack · GitHub

Repository navigation

Fix findbugs RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE in NetworkOrche… - #449

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

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

Conversation

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 commented Jun 14, 2015

Copy link
Copy Markdown

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

asfgit closed this in 0dc2590 Jun 15, 2015
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 closes apache#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>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to 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


Back | FazBrowse Home | New Git URL