rafaelweingartner commented on a change in pull request #2573: Cloudstack 10356
URL: https://github.com/apache/cloudstack/pull/2573#discussion_r182091684
##########
File path:
server/src/main/java/com/cloud/network/router/VpcVirtualNetworkApplianceManagerImpl.java
##########
@@ -740,14 +740,14 @@ public boolean startRemoteAccessVpn(final
RemoteAccessVpn vpn, final VirtualRout
throw new AgentUnavailableException("Unable to send commands to
virtual router ", router.getHostId(), e);
}
Answer answer = cmds.getAnswer("users");
- if (!answer.getResult()) {
+ if (answer != null && !answer.getResult()) {
Review comment:
Sorry, I did not understand your question. Are you asking if I would suggest
to use BooleanUtils here as well? Please, go ahead and do that.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
