Title: [5356] trunk/arch/blackfin/kernel/ptrace.c: revert previous "<=" -> "<" changes ...
Revision
5356
Author
vapier
Date
2008-09-27 11:38:56 -0500 (Sat, 27 Sep 2008)

Log Message

revert previous "<=" -> "<" changes ... we are testing an address + length, not just address

Modified Paths

Diff

Modified: trunk/arch/blackfin/kernel/ptrace.c (5355 => 5356)


--- trunk/arch/blackfin/kernel/ptrace.c	2008-09-26 22:19:58 UTC (rev 5355)
+++ trunk/arch/blackfin/kernel/ptrace.c	2008-09-27 16:38:56 UTC (rev 5356)
@@ -221,22 +221,22 @@
 			pr_debug("ptrace: user address is valid\n");
 
 			if (L1_CODE_LENGTH != 0 && addr >= L1_CODE_START
-			    && addr + sizeof(tmp) < L1_CODE_START + L1_CODE_LENGTH) {
+			    && addr + sizeof(tmp) <= L1_CODE_START + L1_CODE_LENGTH) {
 				safe_dma_memcpy (&tmp, (const void *)(addr), sizeof(tmp));
 				copied = sizeof(tmp);
 
 			} else if (L1_DATA_A_LENGTH != 0 && addr >= L1_DATA_A_START
-			    && addr + sizeof(tmp) < L1_DATA_A_START + L1_DATA_A_LENGTH) {
+			    && addr + sizeof(tmp) <= L1_DATA_A_START + L1_DATA_A_LENGTH) {
 				memcpy(&tmp, (const void *)(addr), sizeof(tmp));
 				copied = sizeof(tmp);
 
 			} else if (L1_DATA_B_LENGTH != 0 && addr >= L1_DATA_B_START
-			    && addr + sizeof(tmp) < L1_DATA_B_START + L1_DATA_B_LENGTH) {
+			    && addr + sizeof(tmp) <= L1_DATA_B_START + L1_DATA_B_LENGTH) {
 				memcpy(&tmp, (const void *)(addr), sizeof(tmp));
 				copied = sizeof(tmp);
 
 			} else if (addr >= FIXED_CODE_START
-			    && addr + sizeof(tmp) < FIXED_CODE_END) {
+			    && addr + sizeof(tmp) <= FIXED_CODE_END) {
 				memcpy(&tmp, (const void *)(addr), sizeof(tmp));
 				copied = sizeof(tmp);
 
@@ -301,22 +301,22 @@
 			pr_debug("ptrace: user address is valid\n");
 
 			if (L1_CODE_LENGTH != 0 && addr >= L1_CODE_START
-			    && addr + sizeof(data) < L1_CODE_START + L1_CODE_LENGTH) {
+			    && addr + sizeof(data) <= L1_CODE_START + L1_CODE_LENGTH) {
 				safe_dma_memcpy ((void *)(addr), &data, sizeof(data));
 				copied = sizeof(data);
 
 			} else if (L1_DATA_A_LENGTH != 0 && addr >= L1_DATA_A_START
-			    && addr + sizeof(data) < L1_DATA_A_START + L1_DATA_A_LENGTH) {
+			    && addr + sizeof(data) <= L1_DATA_A_START + L1_DATA_A_LENGTH) {
 				memcpy((void *)(addr), &data, sizeof(data));
 				copied = sizeof(data);
 
 			} else if (L1_DATA_B_LENGTH != 0 && addr >= L1_DATA_B_START
-			    && addr + sizeof(data) < L1_DATA_B_START + L1_DATA_B_LENGTH) {
+			    && addr + sizeof(data) <= L1_DATA_B_START + L1_DATA_B_LENGTH) {
 				memcpy((void *)(addr), &data, sizeof(data));
 				copied = sizeof(data);
 
 			} else if (addr >= FIXED_CODE_START
-			    && addr + sizeof(data) < FIXED_CODE_END) {
+			    && addr + sizeof(data) <= FIXED_CODE_END) {
 				memcpy((void *)(addr), &data, sizeof(data));
 				copied = sizeof(data);
 
_______________________________________________
Linux-kernel-commits mailing list
[email protected]
http://blackfin.uclinux.org/mailman/listinfo/linux-kernel-commits

Reply via email to