diff options
author | Angel Pons <th3fanbus@gmail.com> | 2021-06-07 13:29:13 +0200 |
---|---|---|
committer | Nico Huber <nico.h@gmx.de> | 2021-06-09 16:07:45 +0000 |
commit | f41d24823c1703e328fc27588bbcf3c96eecdbc9 (patch) | |
tree | 3bd19f60db955c7206fa72a1ccdb7a2607e7001d /physmap.c | |
parent | 3bd47524c044168b01e54e772d23c7fa723dc0d5 (diff) | |
download | flashrom-f41d24823c1703e328fc27588bbcf3c96eecdbc9.tar.gz flashrom-f41d24823c1703e328fc27588bbcf3c96eecdbc9.tar.bz2 flashrom-f41d24823c1703e328fc27588bbcf3c96eecdbc9.zip |
treewide: Drop unnecessary uses of memset/memcpy
Simply provide an initialiser or use a direct assignment instead.
Change-Id: I07385375cd8eec8a95874001b402b2c17ec09e09
Signed-off-by: Angel Pons <th3fanbus@gmail.com>
Reviewed-on: https://review.coreboot.org/c/flashrom/+/55267
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Nico Huber <nico.h@gmx.de>
Diffstat (limited to 'physmap.c')
-rw-r--r-- | physmap.c | 9 |
1 files changed, 3 insertions, 6 deletions
@@ -433,8 +433,7 @@ int wrmsr(int addr, msr_t msr) int setup_cpu_msr(int cpu) { - char msrfilename[64]; - memset(msrfilename, 0, sizeof(msrfilename)); + char msrfilename[64] = { 0 }; snprintf(msrfilename, sizeof(msrfilename), "/dev/cpu/%d/msr", cpu); if (fd_msr != -1) { @@ -509,8 +508,7 @@ int wrmsr(int addr, msr_t msr) int setup_cpu_msr(int cpu) { - char msrfilename[64]; - memset(msrfilename, 0, sizeof(msrfilename)); + char msrfilename[64] = { 0 }; snprintf(msrfilename, sizeof(msrfilename), "/dev/amdmsr"); if (fd_msr != -1) { @@ -591,8 +589,7 @@ int wrmsr(int addr, msr_t msr) int setup_cpu_msr(int cpu) { - char msrfilename[64]; - memset(msrfilename, 0, sizeof(msrfilename)); + char msrfilename[64] = { 0 }; snprintf(msrfilename, sizeof(msrfilename), "/dev/cpu%d", cpu); if (fd_msr != -1) { |