Re: [PATCH 1/1] ASoC: topology: Fix variable overflow in split_format

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, 26 Aug 2019 18:59:16 +0200,
Pierre-Louis Bossart wrote:
> 
> CC: Takashi and Mark.
> 
> On 8/26/19 6:51 AM, Marcin Pietraszko wrote:
> > The 1 which was 32-bit long, when left-shifted by more than 31 caused
> > an overflow, which resulted in some of the formats being not supported.
> 
> Looks like an alsa-lib patch?

Seems so.  I applied to alsa-lib repo now.


thanks,

Takashi


> 
> >
> > Signed-off-by: Marcin Pietraszko <marcin.pietraszko@xxxxxxxxxxxxxxx>
> > ---
> >   src/topology/pcm.c | 2 +-
> >   1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/src/topology/pcm.c b/src/topology/pcm.c
> > index 5f586dc..00a35b9 100644
> > --- a/src/topology/pcm.c
> > +++ b/src/topology/pcm.c
> > @@ -321,7 +321,7 @@ static int split_format(struct snd_soc_tplg_stream_caps *caps, char *str)
> >   			return -EINVAL;
> >   		}
> >   -		caps->formats |= 1 << format;
> > +		caps->formats |= 1ull << format;
> >   		s = strtok(NULL, ", ");
> >   		i++;
> >   	}
> >
> 
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
https://mailman.alsa-project.org/mailman/listinfo/alsa-devel



[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Pulse Audio]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux