|
|
fda493 |
From 21eb397a5fc9227cd95d23e8c74a49cf6a293e57 Mon Sep 17 00:00:00 2001
|
|
|
fda493 |
From: Hari Bathini <hbathini@linux.vnet.ibm.com>
|
|
|
06c2a2 |
Date: Wed, 9 Aug 2017 23:47:42 +0530
|
|
|
fda493 |
Subject: [PATCH] kexec-tools: powerpc: fix command line overflow error
|
|
|
fda493 |
|
|
|
fda493 |
Since kernel commit a5980d064fe2 ("powerpc: Bump COMMAND_LINE_SIZE
|
|
|
fda493 |
to 2048"), powerpc bumped command line size to 2048 but the size
|
|
|
fda493 |
used here is still the default value of 512. Bump it to 2048 to
|
|
|
fda493 |
fix command line overflow errors observed when command line length
|
|
|
fda493 |
is above 512 bytes. Also, get rid of the multiple definitions of
|
|
|
fda493 |
COMMAND_LINE_SIZE macro in ppc architecture.
|
|
|
fda493 |
|
|
|
fda493 |
Signed-off-by: Hari Bathini <hbathini@linux.vnet.ibm.com>
|
|
|
fda493 |
Signed-off-by: Simon Horman <horms@verge.net.au>
|
|
|
06c2a2 |
Signed-off-by: Pingfan Liu <piliu@redhat.com>
|
|
|
fda493 |
---
|
|
|
06c2a2 |
kexec/arch/ppc/crashdump-powerpc.c | 6 +++++-
|
|
|
06c2a2 |
kexec/arch/ppc/crashdump-powerpc.h | 2 --
|
|
|
06c2a2 |
kexec/arch/ppc/fs2dt.c | 1 -
|
|
|
06c2a2 |
kexec/arch/ppc/kexec-ppc.h | 3 ++-
|
|
|
06c2a2 |
kexec/arch/ppc/ops.h | 1 -
|
|
|
06c2a2 |
kexec/arch/ppc64/crashdump-ppc64.c | 6 ++++--
|
|
|
06c2a2 |
kexec/arch/ppc64/crashdump-ppc64.h | 2 +-
|
|
|
06c2a2 |
7 files changed, 12 insertions(+), 9 deletions(-)
|
|
|
fda493 |
|
|
|
06c2a2 |
diff --git a/kexec/arch/ppc/crashdump-powerpc.c b/kexec/arch/ppc/crashdump-powerpc.c
|
|
|
06c2a2 |
index dde6de7..4ad026f 100644
|
|
|
06c2a2 |
--- a/kexec/arch/ppc/crashdump-powerpc.c
|
|
|
06c2a2 |
+++ b/kexec/arch/ppc/crashdump-powerpc.c
|
|
|
06c2a2 |
@@ -252,8 +252,12 @@ static void ulltoa(unsigned long long i, char *str)
|
|
|
06c2a2 |
/* Append str to cmdline */
|
|
|
06c2a2 |
static void add_cmdline(char *cmdline, char *str)
|
|
|
06c2a2 |
{
|
|
|
06c2a2 |
+ int cmdline_size;
|
|
|
06c2a2 |
int cmdlen = strlen(cmdline) + strlen(str);
|
|
|
06c2a2 |
- if (cmdlen > (COMMAND_LINE_SIZE - 1))
|
|
|
06c2a2 |
+
|
|
|
06c2a2 |
+ cmdline_size = (kernel_version() < KERNEL_VERSION(3, 10, 0) ?
|
|
|
06c2a2 |
+ 512 : COMMAND_LINE_SIZE);
|
|
|
06c2a2 |
+ if (cmdlen > (cmdline_size - 1))
|
|
|
06c2a2 |
die("Command line overflow\n");
|
|
|
06c2a2 |
strcat(cmdline, str);
|
|
|
06c2a2 |
}
|
|
|
fda493 |
diff --git a/kexec/arch/ppc/crashdump-powerpc.h b/kexec/arch/ppc/crashdump-powerpc.h
|
|
|
fda493 |
index 9b9b01e..97b5095 100644
|
|
|
fda493 |
--- a/kexec/arch/ppc/crashdump-powerpc.h
|
|
|
fda493 |
+++ b/kexec/arch/ppc/crashdump-powerpc.h
|
|
|
fda493 |
@@ -20,8 +20,6 @@ extern struct arch_options_t arch_options;
|
|
|
fda493 |
#define KERNELBASE PAGE_OFFSET
|
|
|
fda493 |
#define __pa(x) ((unsigned long)(x)-PAGE_OFFSET)
|
|
|
fda493 |
|
|
|
fda493 |
-#define COMMAND_LINE_SIZE 512 /* from kernel */
|
|
|
fda493 |
-
|
|
|
fda493 |
#ifdef CONFIG_BOOKE
|
|
|
fda493 |
/* We don't need backup region in Book E */
|
|
|
fda493 |
#define BACKUP_SRC_START 0x0000
|
|
|
fda493 |
diff --git a/kexec/arch/ppc/fs2dt.c b/kexec/arch/ppc/fs2dt.c
|
|
|
fda493 |
index 6e77379..fed499b 100644
|
|
|
fda493 |
--- a/kexec/arch/ppc/fs2dt.c
|
|
|
fda493 |
+++ b/kexec/arch/ppc/fs2dt.c
|
|
|
fda493 |
@@ -37,7 +37,6 @@
|
|
|
fda493 |
#define TREEWORDS 65536 /* max 32 bit words for properties */
|
|
|
fda493 |
#define MEMRESERVE 256 /* max number of reserved memory blks */
|
|
|
fda493 |
#define MAX_MEMORY_RANGES 1024
|
|
|
fda493 |
-#define COMMAND_LINE_SIZE 512 /* from kernel */
|
|
|
fda493 |
|
|
|
fda493 |
static char pathname[MAXPATH];
|
|
|
fda493 |
static char propnames[NAMESPACE] = { 0 };
|
|
|
fda493 |
diff --git a/kexec/arch/ppc/kexec-ppc.h b/kexec/arch/ppc/kexec-ppc.h
|
|
|
06c2a2 |
index f8fd678..04e728e 100644
|
|
|
fda493 |
--- a/kexec/arch/ppc/kexec-ppc.h
|
|
|
fda493 |
+++ b/kexec/arch/ppc/kexec-ppc.h
|
|
|
fda493 |
@@ -6,6 +6,8 @@
|
|
|
fda493 |
#define CORE_TYPE_ELF32 1
|
|
|
fda493 |
#define CORE_TYPE_ELF64 2
|
|
|
fda493 |
|
|
|
fda493 |
+#define COMMAND_LINE_SIZE 2048 /* from kernel */
|
|
|
fda493 |
+
|
|
|
fda493 |
extern unsigned char setup_simple_start[];
|
|
|
fda493 |
extern uint32_t setup_simple_size;
|
|
|
fda493 |
|
|
|
06c2a2 |
@@ -76,7 +78,6 @@ extern int init_memory_region_info(void);
|
|
|
fda493 |
extern int read_memory_region_limits(int fd, unsigned long long *start,
|
|
|
fda493 |
unsigned long long *end);
|
|
|
06c2a2 |
extern int get_devtree_value(const char *fname, unsigned long long *pvalue);
|
|
|
fda493 |
-#define COMMAND_LINE_SIZE 512 /* from kernel */
|
|
|
fda493 |
/*fs2dt*/
|
|
|
fda493 |
void reserve(unsigned long long where, unsigned long long length);
|
|
|
fda493 |
|
|
|
fda493 |
diff --git a/kexec/arch/ppc/ops.h b/kexec/arch/ppc/ops.h
|
|
|
fda493 |
index 7334a05..5e7a070 100644
|
|
|
fda493 |
--- a/kexec/arch/ppc/ops.h
|
|
|
fda493 |
+++ b/kexec/arch/ppc/ops.h
|
|
|
fda493 |
@@ -12,7 +12,6 @@
|
|
|
fda493 |
#define _PPC_BOOT_OPS_H_
|
|
|
fda493 |
#include "types.h"
|
|
|
fda493 |
|
|
|
fda493 |
-#define COMMAND_LINE_SIZE 512
|
|
|
fda493 |
#define MAX_PATH_LEN 256
|
|
|
fda493 |
#define MAX_PROP_LEN 256 /* What should this be? */
|
|
|
fda493 |
|
|
|
06c2a2 |
diff --git a/kexec/arch/ppc64/crashdump-ppc64.c b/kexec/arch/ppc64/crashdump-ppc64.c
|
|
|
06c2a2 |
index 5a71d51..13995bf 100644
|
|
|
06c2a2 |
--- a/kexec/arch/ppc64/crashdump-ppc64.c
|
|
|
06c2a2 |
+++ b/kexec/arch/ppc64/crashdump-ppc64.c
|
|
|
06c2a2 |
@@ -381,7 +381,7 @@ static void ultoa(uint64_t i, char *str)
|
|
|
06c2a2 |
static int add_cmdline_param(char *cmdline, uint64_t addr, char *cmdstr,
|
|
|
06c2a2 |
char *byte)
|
|
|
06c2a2 |
{
|
|
|
06c2a2 |
- int cmdlen, len, align = 1024;
|
|
|
06c2a2 |
+ int cmdline_size, cmdlen, len, align = 1024;
|
|
|
06c2a2 |
char str[COMMAND_LINE_SIZE], *ptr;
|
|
|
06c2a2 |
|
|
|
06c2a2 |
/* Passing in =xxxK / =xxxM format. Saves space required in cmdline.*/
|
|
|
06c2a2 |
@@ -402,7 +402,9 @@ static int add_cmdline_param(char *cmdline, uint64_t addr, char *cmdstr,
|
|
|
06c2a2 |
strcat(str, byte);
|
|
|
06c2a2 |
len = strlen(str);
|
|
|
06c2a2 |
cmdlen = strlen(cmdline) + len;
|
|
|
06c2a2 |
- if (cmdlen > (COMMAND_LINE_SIZE - 1))
|
|
|
06c2a2 |
+ cmdline_size = (kernel_version() < KERNEL_VERSION(3, 10, 0) ?
|
|
|
06c2a2 |
+ 512 : COMMAND_LINE_SIZE);
|
|
|
06c2a2 |
+ if (cmdlen > (cmdline_size - 1))
|
|
|
06c2a2 |
die("Command line overflow\n");
|
|
|
06c2a2 |
strcat(cmdline, str);
|
|
|
06c2a2 |
dbgprintf("Command line after adding elfcorehdr: %s\n", cmdline);
|
|
|
fda493 |
diff --git a/kexec/arch/ppc64/crashdump-ppc64.h b/kexec/arch/ppc64/crashdump-ppc64.h
|
|
|
fda493 |
index d654c6b..42ccc31 100644
|
|
|
fda493 |
--- a/kexec/arch/ppc64/crashdump-ppc64.h
|
|
|
fda493 |
+++ b/kexec/arch/ppc64/crashdump-ppc64.h
|
|
|
fda493 |
@@ -16,7 +16,7 @@ void add_usable_mem_rgns(unsigned long long base, unsigned long long size);
|
|
|
fda493 |
#define __pa(x) ((unsigned long)(x)-PAGE_OFFSET)
|
|
|
fda493 |
#define MAXMEM (-KERNELBASE-VMALLOCBASE)
|
|
|
fda493 |
|
|
|
fda493 |
-#define COMMAND_LINE_SIZE 512 /* from kernel */
|
|
|
fda493 |
+#define COMMAND_LINE_SIZE 2048 /* from kernel */
|
|
|
fda493 |
/* Backup Region, First 64K of System RAM. */
|
|
|
fda493 |
#define BACKUP_SRC_START 0x0000
|
|
|
fda493 |
#define BACKUP_SRC_END 0xffff
|
|
|
06c2a2 |
--
|
|
|
06c2a2 |
2.7.4
|
|
|
06c2a2 |
|