On Oct 19 2015 20:29, Dan Carpenter wrote: > Let's leave space for the NUL char otherwise the static checkers > complain that we go beyond the end of the array. > > Fixes: 53b3ffee7885 ('ALSA: firewire-tascam: change device probing processing') > Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> Reviewed-by: Takashi Sakamoto <o-takashi@xxxxxxxxxxxxx> Thank you to fix it. Takashi Sakamoto > --- > v2: I truncated the last char in v1 instead of making the buffer larger > > diff --git a/sound/firewire/tascam/tascam.c b/sound/firewire/tascam/tascam.c > index c6747a4..79cb9e9 100644 > --- a/sound/firewire/tascam/tascam.c > +++ b/sound/firewire/tascam/tascam.c > @@ -40,7 +40,7 @@ static int identify_model(struct snd_tscm *tscm) > { > struct fw_device *fw_dev = fw_parent_device(tscm->unit); > const u32 *config_rom = fw_dev->config_rom; > - char model[8]; > + char model[9]; > unsigned int i; > u8 c; > > -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html