Re: [GIT PULL] ASoC updates for v5.8

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

 



On Mon, 01 Jun 2020 23:17:59 +0200,
Takashi Iwai wrote:
> 
> On Mon, 01 Jun 2020 20:44:55 +0200,
> Takashi Iwai wrote:
> > 
> > On Mon, 01 Jun 2020 14:13:23 +0200,
> > Mark Brown wrote:
> > > 
> > > The following changes since commit 9cb1fd0efd195590b828b9b865421ad345a4a145:
> > > 
> > >   Linux 5.7-rc7 (2020-05-24 15:32:54 -0700)
> > > 
> > > are available in the Git repository at:
> > > 
> > >   https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git tags/asoc-v5.8
> > > 
> > > for you to fetch changes up to 358c7c61fd04d324f83d7968daf8dd9a6ff86a9a:
> > > 
> > >   Merge remote-tracking branch 'asoc/for-5.8' into asoc-linus (2020-06-01 13:01:15 +0100)
> > > 
> > > ----------------------------------------------------------------
> > > ASoC: Updates for v5.8
> > > 
> > > This has been another very active release with a bunch of new drivers,
> > > lots of fixes everywhere and continued core improvements from
> > > Morimoto-san:
> > > 
> > >  - Lots of core cleanups and refactorings from Morimoto-san, factoring
> > >    out common operations and making the card abstraction more solid.
> > >  - Continued work on cleaning up and improving the Intel drivers, along
> > >    with some new platform support for them.
> > >  - Fixes to make the Marvell SSPA driver work upstream.
> > >  - Support for AMD Renoir ACP, Dialog DA7212, Freescale EASRC and
> > >    i.MX8M, Intel Elkhard Lake, Maxim MAX98390, Nuvoton NAU8812 and
> > >    NAU8814 and Realtek RT1016.
> > 
> > Pulled now.  Thanks.
> 
> BTW, this pull request caused a compile warning:
>   sound/soc/codecs/max98390.c: In function ‘max98390_dsm_init’:
>   sound/soc/codecs/max98390.c:781:3: warning: format ‘%ld’ expects argument of type ‘long int’, but argument 4 has type ‘size_t {aka const unsigned int}’ [-Wformat=]
> 
> This should be fixed by the correct qualifier 'z' instead of 'l'.

Since no one reacts: below is the quick fix.  I'll resubmit later via
git-send-email.

But, looking more at the code around that line, I could spot many
other bugs.  You cannot trust the firmware file and you must check the
size.  The current code can trigger out-of-bound accesses and crash
very easily when a malformed firmware file is tossed; e.g. just put an
empty file (or a huge file) as dsm_param.bin.

Steve, could you fix it quickly, so that we can cover it for rc1?


thanks,

Takashi

-- 8< --
From: Takashi Iwai <tiwai@xxxxxxx>
Subject: [PATCH] ASoC: max98390: Fix incorrect printf qualifier

This patch addresses a compile warning:
  sound/soc/codecs/max98390.c:781:3: warning: format ‘%ld’ expects argument of type ‘long int’, but argument 4 has type ‘size_t {aka const unsigned int}’ [-Wformat=]

Fixes: a6e3f4f34cdb ("ASoC: max98390: Added Amplifier Driver")
Signed-off-by: Takashi Iwai <tiwai@xxxxxxx>
---
 sound/soc/codecs/max98390.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/sound/soc/codecs/max98390.c b/sound/soc/codecs/max98390.c
index b9ce44dda886..be7cd0aeb6a6 100644
--- a/sound/soc/codecs/max98390.c
+++ b/sound/soc/codecs/max98390.c
@@ -778,7 +778,7 @@ static int max98390_dsm_init(struct snd_soc_component *component)
 	}
 
 	dev_dbg(component->dev,
-		"max98390: param fw size %ld\n",
+		"max98390: param fw size %zd\n",
 		fw->size);
 	dsm_param = (char *)fw->data;
 	dsm_param += MAX98390_DSM_PAYLOAD_OFFSET;
-- 
2.16.4




[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