video/hdmi: prefer strlcpy to strncpy

Started by Nick Desaulniers on May 29, 2018 · 2 messages from 1 author · Last reply May 30, 2018

Subsystems (2)

Maintainers (2)

STALE3088d

open the first message on its own page

Message details and related content
Fixes a stringop-truncation warning from gcc-8.

Signed-off-by: Nick Desaulniers <redacted>
---
 drivers/video/hdmi.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/video/hdmi.c b/drivers/video/hdmi.c
index 111a0ab..46c184c 100644
--- a/drivers/video/hdmi.c
+++ b/drivers/video/hdmi.c
@@ -168,8 +168,8 @@ int hdmi_spd_infoframe_init(struct hdmi_spd_infoframe *frame,
 	frame->version = 1;
 	frame->length = HDMI_SPD_INFOFRAME_SIZE;
 
-	strncpy(frame->vendor, vendor, sizeof(frame->vendor));
-	strncpy(frame->product, product, sizeof(frame->product));
+	strlcpy(frame->vendor, vendor, sizeof(frame->vendor));
+	strlcpy(frame->product, product, sizeof(frame->product));
 
 	return 0;
 }
-- 
2.7.4
Message details and related content
On Mon, May 28, 2018 at 11:59 PM, Nick Desaulniers
[off-list ref] wrote:
quoted hunk
Fixes a stringop-truncation warning from gcc-8.

Signed-off-by: Nick Desaulniers <redacted>
---
 drivers/video/hdmi.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/video/hdmi.c b/drivers/video/hdmi.c
index 111a0ab..46c184c 100644
--- a/drivers/video/hdmi.c
+++ b/drivers/video/hdmi.c
@@ -168,8 +168,8 @@ int hdmi_spd_infoframe_init(struct hdmi_spd_infoframe *frame,
        frame->version = 1;
        frame->length = HDMI_SPD_INFOFRAME_SIZE;

-       strncpy(frame->vendor, vendor, sizeof(frame->vendor));
-       strncpy(frame->product, product, sizeof(frame->product));
+       strlcpy(frame->vendor, vendor, sizeof(frame->vendor));
+       strlcpy(frame->product, product, sizeof(frame->product));

        return 0;
 }
--
2.7.4
Eric points out this can leak kernel memory if size is less than length of src.
Keyboard shortcuts
hback out one level
jnext message in thread / next series-diff position
kprevious message in thread / previous series-diff position
ldrill in
Escclose help / fold thread tree
?toggle this help