The patch titled
off-by-1 in kernel/power/main.c
has been removed from the -mm tree. Its filename is
off-by-1-in-kernel-power-mainc.patch
This patch was probably dropped from -mm because
it has now been merged into a subsystem tree or
into Linus's tree, or because it was folded into
its parent patch in the -mm tree.
From: dean gaudet <dean@xxxxxxxxxx>
There's an off-by-1 in kernel/power/main.c:state_store() ... if your
kernel just happens to have some non-zero data at pm_states[PM_SUSPEND_MAX]
(i.e. one past the end of the array) then it'll let you write anything you
want to /sys/power/state and in response the box will enter S5.
Signed-off-by: dean gaudet <dean@xxxxxxxxxx>
Acked-by: Pavel Machek <pavel@xxxxxx>
Signed-off-by: Andrew Morton <akpm@xxxxxxxx>
---
kernel/power/main.c | 2 +-
1 files changed, 1 insertion(+), 1 deletion(-)
diff -puN kernel/power/main.c~off-by-1-in-kernel-power-mainc kernel/power/main.c
--- devel/kernel/power/main.c~off-by-1-in-kernel-power-mainc 2006-04-27 18:39:03.000000000 -0700
+++ devel-akpm/kernel/power/main.c 2006-04-27 18:39:03.000000000 -0700
@@ -272,7 +272,7 @@ static ssize_t state_store(struct subsys
if (*s && !strncmp(buf, *s, len))
break;
}
- if (*s)
+ if (state < PM_SUSPEND_MAX && *s)
error = enter_state(state);
else
error = -EINVAL;
_
Patches currently in -mm which might be from dean@xxxxxxxxxx are
origin.patch
-
To unsubscribe from this list: send the line "unsubscribe mm-commits" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
