In h_read_settings, val variable could be NULL due to ofnode_read_prop returning an error. This variable would then be used as the src in strcpy.
Add a NULL check after calling ofnode_read_prop. Signed-off-by: Francois Berder <[email protected]> --- boot/cedit.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/boot/cedit.c b/boot/cedit.c index 56dc7c6af15..b1b79d1752a 100644 --- a/boot/cedit.c +++ b/boot/cedit.c @@ -500,6 +500,8 @@ static int h_read_settings(struct scene_obj *obj, void *vpriv) tline = (struct scene_obj_textline *)obj; val = ofnode_read_prop(node, obj->name, &len); + if (!val) + return log_msg_ret("tline", -ENOENT); if (len >= tline->max_chars) return log_msg_ret("str", -ENOSPC); strcpy(abuf_data(&tline->buf), val); -- 2.43.0
