port fix for a segfault: stefan11111/kdrive@0aa5ca4 and use sizeof instead of strlen

Author: stefan11111

kdrive/linux/mouse.c

@@ -386,8 +386,7 @@ static const KmouseProt exps2Prot = {
 #define PSM_4DPLUS_ID           8
 
 static const unsigned char ps2_init[] = {
-	PSMC_ENABLE_DEV,
-	0,
+	PSMC_ENABLE_DEV
 };
 
 #define NINIT_PS2   1
@@ -397,7 +396,6 @@ static const unsigned char wheel_3button_init[] = {
 	PSMC_SET_SAMPLING_RATE, 100,
 	PSMC_SET_SAMPLING_RATE, 80,
 	PSMC_SEND_DEV_ID,
-	0,
 };
 
 #define NINIT_IMPS2 4
@@ -410,7 +408,6 @@ static const unsigned char wheel_5button_init[] = {
 	PSMC_SET_SAMPLING_RATE, 200,
 	PSMC_SET_SAMPLING_RATE, 80,
 	PSMC_SEND_DEV_ID,
-	0
 };
 
 #define NINIT_EXPS2 7
@@ -419,7 +416,6 @@ static const unsigned char intelli_init[] = {
 	PSMC_SET_SAMPLING_RATE, 200,
 	PSMC_SET_SAMPLING_RATE, 100,
 	PSMC_SET_SAMPLING_RATE, 80,
-	0
 };
 
 #define NINIT_INTELLI	3
@@ -458,7 +454,7 @@ static Bool ps2Init(KdMouseInfo * mi)
 	int ninit;
 
 	/* Send Intellimouse initialization sequence */
-	MouseWriteBytes(km->iob.fd, intelli_init, strlen((char *)intelli_init),
+	MouseWriteBytes(km->iob.fd, intelli_init, sizeof(intelli_init),
 			100);
 	/*
 	 * Send ID command
@@ -484,7 +480,7 @@ static Bool ps2Init(KdMouseInfo * mi)
 		break;
 	}
 	if (init)
-		MouseWriteBytes(km->iob.fd, init, strlen((char *)init), 100);
+		MouseWriteBytes(km->iob.fd, init, sizeof(init), 100);
 	/*
 	 * Flush out the available data to eliminate responses to the
 	 * initialization string.  Make sure any partial event is
@@ -729,6 +725,9 @@ static void MouseFirstProtocol(Kmouse * km, char *prot)
 			for (i = 0; i < NUM_PROT; i++)
 				ErrorF(" %s", kmouseProts[i]->name);
 			ErrorF("\n");
+			km->i_prot = 0;
+			km->prot = kmouseProts[km->i_prot];
+			ErrorF("Falling back to %s\n", km->prot->name);
 		} else {
 			km->prot = kmouseProts[km->i_prot];
 			if (km->tty && !km->prot->tty)
@@ -754,7 +753,7 @@ static void MouseNextProtocol(Kmouse * km)
 	do {
 		if (!km->prot)
 			km->i_prot = 0;
-		else if (++km->i_prot == NUM_PROT)
+		else if (++km->i_prot >= NUM_PROT)
 			km->i_prot = 0;
 		km->prot = kmouseProts[km->i_prot];
 	} while (km->prot->tty != km->tty);