[PATCH] kexec ppc64: fix misaligned cmdline
Michael Neuling
mikey at neuling.org
Mon Jun 4 03:23:45 EDT 2007
If the cmdline changes between boots, we can get misalignment of the
bootargs entry, which in turn corrupts our device tree blob and hence
kills our kexec boot.
Specifically, if the cmdline length was >= 8 before and the new cmdline
length is < 8, we can get corruption.
Signed-off-by: Michael Neuling <mikey at neuling.org>
---
kexec/arch/ppc64/fs2dt.c | 7 ++++++-
1 file changed, 6 insertions(+), 1 deletion(-)
Index: kexec-tools-testing/kexec/arch/ppc64/fs2dt.c
===================================================================
--- kexec-tools-testing.orig/kexec/arch/ppc64/fs2dt.c
+++ kexec-tools-testing/kexec/arch/ppc64/fs2dt.c
@@ -197,6 +197,7 @@ static void putprops(char *fn, struct di
struct dirent *dp;
int i = 0, fd, len;
struct stat statbuf;
+ int dt_realigned = 0;
for (i = 0; i < numlist; i++) {
dp = nlist[i];
@@ -243,8 +244,10 @@ static void putprops(char *fn, struct di
*dt++ = len;
*dt++ = propnum(fn);
- if ((len >= 8) && ((unsigned long)dt & 0x4))
+ if ((len >= 8) && ((unsigned long)dt & 0x4)){
dt++;
+ dt_realigned = 1;
+ }
fd = open(pathname, O_RDONLY);
if (fd == -1)
@@ -283,6 +286,8 @@ static void putprops(char *fn, struct di
strcat(local_cmdline, " ");
cmd_len = strlen(local_cmdline);
cmd_len = cmd_len + 1;
+ if (dt_realigned && cmd_len < 8)
+ dt--;
memcpy(dt, local_cmdline,cmd_len);
len = cmd_len;
*dt_len = cmd_len;
More information about the kexec
mailing list