Fixed remaining getter/setter return checks.

This commit is contained in:
Armin Novak
2019-05-09 13:50:39 +02:00
parent 93bf375686
commit 4d4e7a69d7
2 changed files with 7 additions and 4 deletions

View File

@@ -1009,8 +1009,8 @@ BOOL freerdp_client_populate_settings_from_rdp_file(rdpFile* file, rdpSettings*
if (~((size_t) file->RemoteApplicationFile))
{
if (freerdp_settings_set_string(settings, FreeRDP_RemoteApplicationGuid,
file->RemoteApplicationGuid) != 0)
if (!freerdp_settings_set_string(settings, FreeRDP_RemoteApplicationGuid,
file->RemoteApplicationGuid))
return FALSE;
}
@@ -1168,7 +1168,8 @@ BOOL freerdp_client_populate_settings_from_rdp_file(rdpFile* file, rdpSettings*
* devicestoredirect:s:USB\VID_04A9&PID_30C1\6&4BD985D&0&2;,DynamicDevices
*
*/
freerdp_settings_set_bool(settings, FreeRDP_RedirectDrives, TRUE);
if (!freerdp_settings_set_bool(settings, FreeRDP_RedirectDrives, TRUE))
return FALSE;
}
if (~((size_t) file->DrivesToRedirect))

View File

@@ -1155,7 +1155,9 @@ BOOL freerdp_assistance_populate_settings_from_assistance_file(rdpAssistanceFile
rdpSettings* settings)
{
UINT32 i;
freerdp_settings_set_bool(settings, FreeRDP_RemoteAssistanceMode, TRUE);
if (!freerdp_settings_set_bool(settings, FreeRDP_RemoteAssistanceMode, TRUE))
return FALSE;
if (!file->RASessionId || !file->MachineAddresses)
return FALSE;