From: Xu Panda <xu.pa...@zte.com.cn>

The implementation of strscpy() is more robust and safer.
That's now the recommended way to copy NUL-terminated strings.

Signed-off-by: Xu Panda <xu.pa...@zte.com.cn>
Signed-off-by: Yang Yang <yang.yan...@zte.com>
---
 drivers/video/fbdev/aty/atyfb_base.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/drivers/video/fbdev/aty/atyfb_base.c 
b/drivers/video/fbdev/aty/atyfb_base.c
index 0ccf5d401ecb..851c1236fddb 100644
--- a/drivers/video/fbdev/aty/atyfb_base.c
+++ b/drivers/video/fbdev/aty/atyfb_base.c
@@ -3192,8 +3192,7 @@ static void aty_init_lcd(struct atyfb_par *par, u32 
bios_base)
                 * which we print to the screen.
                 */
                id = *(u8 *)par->lcd_table;
-               strncpy(model, (char *)par->lcd_table+1, 24);
-               model[23] = 0;
+               strscpy(model, (char *)par->lcd_table+1, 24);

                width = par->lcd_width = *(u16 *)(par->lcd_table+25);
                height = par->lcd_height = *(u16 *)(par->lcd_table+27);
-- 
2.15.2

Reply via email to