# HG changeset patch # User tron # Date 1150291350 0 # Node ID cccfb075a147523da1826239025870a75ce9a014 # Parent a2dfcfd71652b918dd1d68484cf03edc268f6007 (svn r5265) Replace a dozen bogus uses of strncmp() by strcmp() diff --git a/console_cmds.c b/console_cmds.c --- a/console_cmds.c +++ b/console_cmds.c @@ -1202,7 +1202,7 @@ DEF_CONSOLE_HOOK(ConHookServerPW) { - if (strncmp(_network_server_password, "*", NETWORK_PASSWORD_LENGTH) == 0) { + if (strcmp(_network_server_password, "*") == 0) { _network_server_password[0] = '\0'; _network_game_info.use_password = 0; } else { @@ -1215,7 +1215,7 @@ DEF_CONSOLE_HOOK(ConHookRconPW) { - if (strncmp(_network_rcon_password, "*", NETWORK_PASSWORD_LENGTH) == 0) + if (strcmp(_network_rcon_password, "*") == 0) _network_rcon_password[0] = '\0'; ttd_strlcpy(_network_game_info.rcon_password, _network_rcon_password, sizeof(_network_game_info.rcon_password)); @@ -1239,8 +1239,7 @@ if (argc != 1) return false; - if (strncmp(argv[0], "*", sizeof(_network_player_info[_local_player].password)) == 0) - argv[0][0] = '\0'; + if (strcmp(argv[0], "*") == 0) argv[0][0] = '\0'; ttd_strlcpy(_network_player_info[_local_player].password, argv[0], sizeof(_network_player_info[_local_player].password)); diff --git a/network_server.c b/network_server.c --- a/network_server.c +++ b/network_server.c @@ -578,8 +578,8 @@ #if defined(WITH_REV) || defined(WITH_REV_HACK) // Check if the client has revision control enabled - if (strncmp(NOREV_STRING, client_revision, sizeof(client_revision)) != 0) { - if (strncmp(_network_game_info.server_revision, client_revision, sizeof(_network_game_info.server_revision) - 1) != 0) { + if (strcmp(NOREV_STRING, client_revision) != 0) { + if (strcmp(_network_game_info.server_revision, client_revision) != 0) { // Different revisions!! SEND_COMMAND(PACKET_SERVER_ERROR)(cs, NETWORK_ERROR_WRONG_REVISION); @@ -656,7 +656,7 @@ if (cs->status == STATUS_INACTIVE && type == NETWORK_GAME_PASSWORD) { // Check game-password - if (strncmp(password, _network_game_info.server_password, sizeof(password)) != 0) { + if (strcmp(password, _network_game_info.server_password) != 0) { // Password is invalid SEND_COMMAND(PACKET_SERVER_ERROR)(cs, NETWORK_ERROR_WRONG_PASSWORD); return; @@ -675,7 +675,7 @@ } else if (cs->status == STATUS_INACTIVE && type == NETWORK_COMPANY_PASSWORD) { ci = DEREF_CLIENT_INFO(cs); - if (strncmp(password, _network_player_info[ci->client_playas - 1].password, sizeof(password)) != 0) { + if (strcmp(password, _network_player_info[ci->client_playas - 1].password) != 0) { // Password is invalid SEND_COMMAND(PACKET_SERVER_ERROR)(cs, NETWORK_ERROR_WRONG_PASSWORD); return; @@ -1119,7 +1119,7 @@ NetworkRecv_string(cs, p, pass, sizeof(pass)); NetworkRecv_string(cs, p, command, sizeof(command)); - if (strncmp(pass, _network_game_info.rcon_password, sizeof(pass)) != 0) { + if (strcmp(pass, _network_game_info.rcon_password) != 0) { DEBUG(net, 0)("[RCon] Wrong password from client-id %d", cs->index); return; } @@ -1396,7 +1396,7 @@ found_name = true; FOR_ALL_CLIENTS(new_cs) { ci = DEREF_CLIENT_INFO(new_cs); - if (strncmp(ci->client_name, new_name, NETWORK_CLIENT_NAME_LENGTH) == 0) { + if (strcmp(ci->client_name, new_name) == 0) { // Name already in use found_name = false; break; @@ -1405,7 +1405,7 @@ // Check if it is the same as the server-name ci = NetworkFindClientInfoFromIndex(NETWORK_SERVER_INDEX); if (ci != NULL) { - if (strncmp(ci->client_name, new_name, NETWORK_CLIENT_NAME_LENGTH) == 0) { + if (strcmp(ci->client_name, new_name) == 0) { // Name already in use found_name = false; } diff --git a/network_udp.c b/network_udp.c --- a/network_udp.c +++ b/network_udp.c @@ -142,8 +142,8 @@ /* Check if we are allowed on this server based on the revision-match */ item->info.compatible = ( - strncmp(item->info.server_revision, _openttd_revision, NETWORK_REVISION_LENGTH) == 0 || - strncmp(item->info.server_revision, NOREV_STRING, NETWORK_REVISION_LENGTH) == 0) ? true : false; + strcmp(item->info.server_revision, _openttd_revision) == 0 || + strcmp(item->info.server_revision, NOREV_STRING) == 0) ? true : false; break; }