Le 14/12/2023 à 06:55, Nicholas Miehlbradt a écrit : > All elements of bounce_buffer are eventually read and passed to the > hypervisor so it should probably be fully initialized.
should or shall ? > > Signed-off-by: Nicholas Miehlbradt <nicho...@linux.ibm.com> Should be a Fixed: tag ? > --- > drivers/tty/hvc/hvc_vio.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/tty/hvc/hvc_vio.c b/drivers/tty/hvc/hvc_vio.c > index 736b230f5ec0..1e88bfcdde20 100644 > --- a/drivers/tty/hvc/hvc_vio.c > +++ b/drivers/tty/hvc/hvc_vio.c > @@ -227,7 +227,7 @@ static const struct hv_ops hvterm_hvsi_ops = { > static void udbg_hvc_putc(char c) > { > int count = -1; > - unsigned char bounce_buffer[16]; > + unsigned char bounce_buffer[16] = { 0 }; Why 16 while we have a count of 1 in the call to hvterm_raw_put_chars() ? > > if (!hvterm_privs[0]) > return;