Mercurial > hg > openttd
changeset 4026:cccfb075a147 draft
(svn r5265) Replace a dozen bogus uses of strncmp() by strcmp()
author | tron <tron@openttd.org> |
---|---|
date | Wed, 14 Jun 2006 13:22:30 +0000 |
parents | a2dfcfd71652 |
children | 029780a1439d |
files | console_cmds.c network_server.c network_udp.c |
diffstat | 3 files changed, 12 insertions(+), 13 deletions(-) [+] |
line wrap: on
line diff
--- 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));
--- 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; }
--- 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; }