(svn r5265) Replace a dozen bogus uses of strncmp() by strcmp()
authortron
Wed, 14 Jun 2006 13:22:30 +0000
changeset 4026 f6faccd92b1c
parent 4025 489430d5cdec
child 4027 22c471987aa3
(svn r5265) Replace a dozen bogus uses of strncmp() by strcmp()
console_cmds.c
network_server.c
network_udp.c
--- a/console_cmds.c	Wed Jun 14 12:30:31 2006 +0000
+++ b/console_cmds.c	Wed Jun 14 13:22:30 2006 +0000
@@ -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	Wed Jun 14 12:30:31 2006 +0000
+++ b/network_server.c	Wed Jun 14 13:22:30 2006 +0000
@@ -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	Wed Jun 14 12:30:31 2006 +0000
+++ b/network_udp.c	Wed Jun 14 13:22:30 2006 +0000
@@ -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;
 	}