Hi, On 9/22/23 12:50, Hans Verkuil wrote: > Fixes these compiler warnings: > > drivers/staging/media/atomisp/pci/runtime/debug/src/ia_css_debug.c: In function 'ia_css_debug_pipe_graph_dump_stage': > drivers/staging/media/atomisp/pci/runtime/debug/src/ia_css_debug.c:2786:66: warning: '\n' directive output may be truncated writing 2 bytes into a region of size between 0 and 198 [-Wformat-truncation=] > 2786 | "%s\\n%s\\n%s", > | ^~~ > drivers/staging/media/atomisp/pci/runtime/debug/src/ia_css_debug.c:2785:49: note: 'snprintf' output between 5 and 302 bytes into a destination of size 200 > 2785 | snprintf(enable_info, sizeof(enable_info), > | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > 2786 | "%s\\n%s\\n%s", > | ~~~~~~~~~~~~~~~ > 2787 | enable_info1, enable_info2, > | ~~~~~~~~~~~~~~~~~~~~~~~~~~~ > 2788 | enable_info3); > | ~~~~~~~~~~~~~ > drivers/staging/media/atomisp/pci/runtime/debug/src/ia_css_debug.c:2772:66: warning: '\n' directive output may be truncated writing 2 bytes into a region of size between 0 and 198 [-Wformat-truncation=] > 2772 | "%s\\n%s\\n%s", > | ^~~ > drivers/staging/media/atomisp/pci/runtime/debug/src/ia_css_debug.c:2771:49: note: 'snprintf' output between 5 and 302 bytes into a destination of size 200 > 2771 | snprintf(enable_info, sizeof(enable_info), > | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > 2772 | "%s\\n%s\\n%s", > | ~~~~~~~~~~~~~~~ > 2773 | enable_info1, enable_info2, > | ~~~~~~~~~~~~~~~~~~~~~~~~~~~ > 2774 | enable_info3); > | ~~~~~~~~~~~~~ > drivers/staging/media/atomisp/pci/runtime/debug/src/ia_css_debug.c:2749:92: warning: 'snprintf' output may be truncated before the last format character [-Wformat-truncation=] > 2749 | snprintf(enable_info, sizeof(enable_info), "%s\\n%s", > | ^ > drivers/staging/media/atomisp/pci/runtime/debug/src/ia_css_debug.c:2749:41: note: 'snprintf' output between 3 and 201 bytes into a destination of size 200 > 2749 | snprintf(enable_info, sizeof(enable_info), "%s\\n%s", > | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > 2750 | enable_info1, enable_info2); > | ~~~~~~~~~~~~~~~~~~~~~~~~~~~ > > Signed-off-by: Hans Verkuil <hverkuil-cisco@xxxxxxxxx> > Cc: Hans de Goede <hdegoede@xxxxxxxxxx> Thanks, patch looks good to me: Reviewed-by: Hans de Goede <hdegoede@xxxxxxxxxx> Regards, Hans > --- > .../staging/media/atomisp/pci/runtime/debug/src/ia_css_debug.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/staging/media/atomisp/pci/runtime/debug/src/ia_css_debug.c b/drivers/staging/media/atomisp/pci/runtime/debug/src/ia_css_debug.c > index bb6204cb42c5..8b0251073f93 100644 > --- a/drivers/staging/media/atomisp/pci/runtime/debug/src/ia_css_debug.c > +++ b/drivers/staging/media/atomisp/pci/runtime/debug/src/ia_css_debug.c > @@ -2673,7 +2673,7 @@ ia_css_debug_pipe_graph_dump_stage( > char enable_info1[100]; > char enable_info2[100]; > char enable_info3[100]; > - char enable_info[200]; > + char enable_info[302]; > struct ia_css_binary_info *bi = stage->binary_info; > > /* Split it in 2 function-calls to keep the amount of
