clk: zynqmp: Replaced strncpy() with strscpy()
authorShubhrajyoti Datta <shubhrajyoti.datta@xilinx.com>
Tue, 10 May 2022 07:01:53 +0000 (12:31 +0530)
committerStephen Boyd <sboyd@kernel.org>
Tue, 23 Aug 2022 00:01:05 +0000 (17:01 -0700)
Replaced strncpy() with strscpy() as the clock names are supposed to
be NULL terminated.

Signed-off-by: Shubhrajyoti Datta <shubhrajyoti.datta@xilinx.com>
Link: https://lore.kernel.org/r/20220510070154.29528-2-shubhrajyoti.datta@xilinx.com
Acked-by: Michal Simek <michal.simek@amd.com>
Signed-off-by: Stephen Boyd <sboyd@kernel.org>
drivers/clk/zynqmp/clkc.c

index 2c9da6623b84e66e376b90ace79658d3e848bfd4..543e6e81712fb4456f737d228afcce7f6892d602 100644 (file)
@@ -163,7 +163,7 @@ static int zynqmp_get_clock_name(u32 clk_id, char *clk_name)
 
        ret = zynqmp_is_valid_clock(clk_id);
        if (ret == 1) {
-               strncpy(clk_name, clock[clk_id].clk_name, MAX_NAME_LEN);
+               strscpy(clk_name, clock[clk_id].clk_name, MAX_NAME_LEN);
                return 0;
        }
 
@@ -719,7 +719,7 @@ static void zynqmp_get_clock_info(void)
 
                if (!strcmp(name.name, RESERVED_CLK_NAME))
                        continue;
-               strncpy(clock[i].clk_name, name.name, MAX_NAME_LEN);
+               strscpy(clock[i].clk_name, name.name, MAX_NAME_LEN);
        }
 
        /* Get topology of all clock */