[PATCH] ALSA: hwdep: fix a left shifting 1 by 31 UB bug
Takashi Iwai
tiwai at suse.de
Tue May 26 18:11:10 CEST 2020
On Tue, 26 May 2020 18:06:02 +0200,
Changming Liu wrote:
>
> > From: Takashi Iwai <tiwai at suse.de>
> > Sent: Tuesday, May 26, 2020 2:22 AM
> > To: Changming Liu <liu.changm at northeastern.edu>
> > Cc: perex at perex.cz; alsa-devel at alsa-project.org
> > Subject: Re: [PATCH] ALSA: hwdep: fix a left shifting 1 by 31 UB bug
> >
> > On Tue, 26 May 2020 02:39:21 +0200,
> > Changming Liu wrote:
> > >
> > > The "info.index" variable can be 31 in "1 << info.index".
> > > This might trigger an undefined behavior since 1 is signed.
> > >
> > > Fix this by casting 1 to 1u just to be sure "1u << 31" is defined.
> > >
> > > Signed-off-by: Changming Liu <liu.changm at northeastern.edu>
> >
> > Thanks for the patch.
> >
> > Unfortunately it seems that your MUA modified in quoted-printable and
> > inapplicable with git-am as is. Since the changes are so small, I
> > manually applied it now.
> >
> I see, thank you very much for pointing out this problem,
> I'll never use my MUA to send patch again.
> Also thank you for this manual modification.
>
> > Please fix the setup of your MUA at the next time, or better to try
> > git-send-email to submit directly.
> >
> Sorry that I cannot configure my university email to work on git-send-email,
> I'll use my gmail to send patch via git next time.
That should work. In the worst case, use an attachment.
thanks,
Takashi
More information about the Alsa-devel
mailing list