diff options
| author | Rickard Strandqvist <rickard_strandqvist@spectrumdigital.se> | 2014-09-14 18:02:12 +0200 |
|---|---|---|
| committer | Helge Deller <deller@gmx.de> | 2014-09-21 21:07:48 +0200 |
| commit | 94c457deff2a211f8372f69a4d7b0d288183756a (patch) | |
| tree | 3c697069758c133e549a76f899ebfc6fd89cb5e2 | |
| parent | parisc: ptrace: use secure_computing_strict() (diff) | |
| download | linux-94c457deff2a211f8372f69a4d7b0d288183756a.tar.xz linux-94c457deff2a211f8372f69a4d7b0d288183756a.zip | |
parisc: pdc_stable.c: Cleaning up unnecessary use of memset in conjunction with strncpy
Using memset before strncpy just to ensure a trailing null character is
an unnecessary double writing of a string
Patch modified by Helge Deller to additionally reduce stack usage.
Signed-off-by: Rickard Strandqvist <rickard_strandqvist@spectrumdigital.se>
Signed-off-by: Helge Deller <deller@gmx.de>
| -rw-r--r-- | drivers/parisc/pdc_stable.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/drivers/parisc/pdc_stable.c b/drivers/parisc/pdc_stable.c index 0f54ab6260df..e4b73c2a053a 100644 --- a/drivers/parisc/pdc_stable.c +++ b/drivers/parisc/pdc_stable.c @@ -755,7 +755,7 @@ static ssize_t pdcs_auto_write(struct kobject *kobj, { struct pdcspath_entry *pathentry; unsigned char flags; - char in[count+1], *temp; + char in[8], *temp; char c; if (!capable(CAP_SYS_ADMIN)) @@ -765,8 +765,9 @@ static ssize_t pdcs_auto_write(struct kobject *kobj, return -EINVAL; /* We'll use a local copy of buf */ - memset(in, 0, count+1); + count = min_t(size_t, count, 7); strncpy(in, buf, count); + in[count] = '\0'; /* Current flags are stored in primary boot path entry */ pathentry = &pdcspath_entry_primary; |