AMajor/SoftEtherVPN
1
0
Fork 0
mirror of https://github.com/SoftEtherVPN/SoftEtherVPN.git synced 2024-11-22 17:39:53 +03:00
Code Releases Activity

src/Cedar/Client.c: remove redundant "if" statement, unused variable

Browse Source 
found by coverity, cppcheck

[src/Cedar/Client.c:9094]: (style) Unused variable: i
[src/Cedar/Client.c:500] -> [src/Cedar/Client.c:503]: (style) Variable 'ret' is reassigned a value before the old one has been used.
This commit is contained in:
Ilya Shipitsin 2018-08-22 20:58:03 +05:00
parent e8c6e2c1d9
commit a2d5fb7015
1 changed files with 5 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

21
src/Cedar/Client.c
View File

@ -497,13 +497,11 @@ bool CnIsCnServiceReady()
// Check whether the notification service is already running
bool CnCheckAlreadyExists(bool lock)
{
bool ret = false;
#ifdef OS_WIN32
ret = Win32CnCheckAlreadyExists(lock);
return Win32CnCheckAlreadyExists(lock);
#else
return false;
#endif
return ret;
}
typedef struct CNC_STATUS_PRINTER_WINDOW_PARAM
@ -9091,7 +9089,6 @@ void CiSetVLanToDefault(CLIENT *c)
}
#else // OS_WIN32
{
UINT i;
UNIX_VLAN *v;
LockList(c->UnixVLanList);
@ -9163,16 +9160,8 @@ void CiInitConfiguration(CLIENT *c)
// Clear the password
Hash(c->EncryptedPassword, "", 0, true);
// Initialize the client configuration
if (OS_IS_WINDOWS(GetOsInfo()->OsType))
{
// Disable remote management in Windows
c->Config.AllowRemoteConfig = false;
}
else
{
// Disable the remote management also in case of UNIX
c->Config.AllowRemoteConfig = false;
}
// Disable remote management
c->Config.AllowRemoteConfig = false;
StrCpy(c->Config.KeepConnectHost, sizeof(c->Config.KeepConnectHost), CLIENT_DEFAULT_KEEPALIVE_HOST);
c->Config.KeepConnectPort = CLIENT_DEFAULT_KEEPALIVE_PORT;
c->Config.KeepConnectProtocol = CONNECTION_UDP;