diff options
| author | Brian Norris <computersforpeace@gmail.com> | 2015-12-09 23:50:26 +0100 |
|---|---|---|
| committer | Shuah Khan <shuahkh@osg.samsung.com> | 2016-01-07 21:43:28 +0100 |
| commit | be4a1326d12cce8df1f57017bf4112eaab437a38 (patch) | |
| tree | c271e4c6942f46a387efabcdf3d003c9a80b5084 /lib | |
| parent | test: firmware_class: report errors properly on failure (diff) | |
| download | linux-be4a1326d12cce8df1f57017bf4112eaab437a38.tar.xz linux-be4a1326d12cce8df1f57017bf4112eaab437a38.zip | |
test: firmware_class: use kstrndup() where appropriate
We're essentially just doing an open-coded kstrndup(). The only
differences are with what happens after the first '\0' character, but
request_firmware() doesn't care about that.
Suggested-by: Kees Cook <keescook@chromium.org>
Signed-off-by: Brian Norris <computersforpeace@gmail.com>
Signed-off-by: Shuah Khan <shuahkh@osg.samsung.com>
Diffstat (limited to 'lib')
| -rw-r--r-- | lib/test_firmware.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/lib/test_firmware.c b/lib/test_firmware.c index 841191061816..690b9c35a274 100644 --- a/lib/test_firmware.c +++ b/lib/test_firmware.c @@ -54,10 +54,9 @@ static ssize_t trigger_request_store(struct device *dev, int rc; char *name; - name = kzalloc(count + 1, GFP_KERNEL); + name = kstrndup(buf, count, GFP_KERNEL); if (!name) return -ENOSPC; - memcpy(name, buf, count); pr_info("loading '%s'\n", name); |