diff options
| author | Heiko Carstens <heiko.carstens@de.ibm.com> | 2011-03-23 10:14:59 +0100 |
|---|---|---|
| committer | Martin Schwidefsky <sky@mschwide.boeblingen.de.ibm.com> | 2011-03-23 10:15:14 +0100 |
| commit | 894e491e427ae40af62b871e05f9a3f80bbddb6c (patch) | |
| tree | 176c995f2d50d0b65c75fbfe34e05d66a87882ef /arch/s390 | |
| parent | [S390] early: Fix possible overlapping data buffer (diff) | |
| download | linux-894e491e427ae40af62b871e05f9a3f80bbddb6c.tar.xz linux-894e491e427ae40af62b871e05f9a3f80bbddb6c.zip | |
[S390] early: limit savesys cmd string handling
Use snprintf() here as well so we won't have to deal with this again.
Signed-off-by: Heiko Carstens <heiko.carstens@de.ibm.com>
Signed-off-by: Martin Schwidefsky <schwidefsky@de.ibm.com>
Diffstat (limited to 'arch/s390')
| -rw-r--r-- | arch/s390/kernel/early.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/arch/s390/kernel/early.c b/arch/s390/kernel/early.c index 668138ee85d9..068f8465c4ee 100644 --- a/arch/s390/kernel/early.c +++ b/arch/s390/kernel/early.c @@ -143,8 +143,9 @@ static noinline __init void create_kernel_nss(void) snprintf(defsys_cmd + hlen, DEFSYS_CMD_SIZE - hlen, " EW MINSIZE=%.7iK PARMREGS=0-13", min_size); defsys_cmd[DEFSYS_CMD_SIZE - 1] = '\0'; - sprintf(savesys_cmd, "SAVESYS %s \n IPL %s", - kernel_nss_name, kernel_nss_name); + snprintf(savesys_cmd, SAVESYS_CMD_SIZE, "SAVESYS %s \n IPL %s", + kernel_nss_name, kernel_nss_name); + savesys_cmd[SAVESYS_CMD_SIZE - 1] = '\0'; __cpcmd(defsys_cmd, NULL, 0, &response); |