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>
Original-Reviewed-on: https://review.coreboot.org/c/flashrom/+/55267
Original-Reviewed-by: Nico Huber <nico.h@gmx.de>
Reviewed-on: https://review.coreboot.org/c/flashrom-stable/+/71372
Reviewed-by: Nico Huber <nico.h@gmx.de>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
diff --git a/physmap.c b/physmap.c
index 6c36814..796860c 100644
--- a/physmap.c
+++ b/physmap.c
@@ -433,8 +433,7 @@
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 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 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) {