linux-kernel - Re: [PATCH] ASoC: wm_adsp: Don't overrun firmware file buffer when reading region data

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <201612200355.immIPrXm%fengguang.wu@intel.com>
Date:   Tue, 20 Dec 2016 03:20:03 +0800
From:   kbuild test robot <lkp@...el.com>
To:     Richard Fitzgerald <rf@...nsource.wolfsonmicro.com>
Cc:     kbuild-all@...org, broonie@...nel.org,
        linux-kernel@...r.kernel.org, alsa-devel@...a-project.org,
        patches@...nsource.wolfsonmicro.com
Subject: Re: [PATCH] ASoC: wm_adsp: Don't overrun firmware file buffer when
 reading region data

Hi Richard,

[auto build test WARNING on asoc/for-next]
[also build test WARNING on v4.9 next-20161219]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]

url:    https://github.com/0day-ci/linux/commits/Richard-Fitzgerald/ASoC-wm_adsp-Don-t-overrun-firmware-file-buffer-when-reading-region-data/20161220-021733
base:   https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git for-next
config: sparc64-allmodconfig (attached as .config)
compiler: sparc64-linux-gnu-gcc (Debian 6.1.1-9) 6.1.1 20160705
reproduce:
        wget https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp-tests.git/plain/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # save the attached .config to linux build tree
        make.cross ARCH=sparc64 

All warnings (new ones prefixed by >>):

   sound/soc/codecs/wm_adsp.c: In function 'wm_adsp_load':
>> sound/soc/codecs/wm_adsp.c:40:21: warning: format '%d' expects argument of type 'int', but argument 8 has type 'size_t {aka const long unsigned int}' [-Wformat=]
     dev_err(_dsp->dev, "DSP%d: " fmt, _dsp->num, ##__VA_ARGS__)
                        ^
>> sound/soc/codecs/wm_adsp.c:1705:4: note: in expansion of macro 'adsp_err'
       adsp_err(dsp,
       ^~~~~~~~
   sound/soc/codecs/wm_adsp.c: In function 'wm_adsp_load_coeff':
>> sound/soc/codecs/wm_adsp.c:40:21: warning: format '%d' expects argument of type 'int', but argument 8 has type 'size_t {aka const long unsigned int}' [-Wformat=]
     dev_err(_dsp->dev, "DSP%d: " fmt, _dsp->num, ##__VA_ARGS__)
                        ^
   sound/soc/codecs/wm_adsp.c:2250:5: note: in expansion of macro 'adsp_err'
        adsp_err(dsp,
        ^~~~~~~~

vim +40 sound/soc/codecs/wm_adsp.c

cdcd7f72 Charles Keepax      2014-11-14  24  #include <linux/vmalloc.h>
6ab2b7b4 Dimitris Papastamos 2013-05-08  25  #include <linux/workqueue.h>
f9f55e31 Richard Fitzgerald  2015-06-11  26  #include <linux/debugfs.h>
2159ad93 Mark Brown          2012-10-11  27  #include <sound/core.h>
2159ad93 Mark Brown          2012-10-11  28  #include <sound/pcm.h>
2159ad93 Mark Brown          2012-10-11  29  #include <sound/pcm_params.h>
2159ad93 Mark Brown          2012-10-11  30  #include <sound/soc.h>
2159ad93 Mark Brown          2012-10-11  31  #include <sound/jack.h>
2159ad93 Mark Brown          2012-10-11  32  #include <sound/initval.h>
2159ad93 Mark Brown          2012-10-11  33  #include <sound/tlv.h>
2159ad93 Mark Brown          2012-10-11  34  
2159ad93 Mark Brown          2012-10-11  35  #include "wm_adsp.h"
2159ad93 Mark Brown          2012-10-11  36  
2159ad93 Mark Brown          2012-10-11  37  #define adsp_crit(_dsp, fmt, ...) \
2159ad93 Mark Brown          2012-10-11  38  	dev_crit(_dsp->dev, "DSP%d: " fmt, _dsp->num, ##__VA_ARGS__)
2159ad93 Mark Brown          2012-10-11  39  #define adsp_err(_dsp, fmt, ...) \
2159ad93 Mark Brown          2012-10-11 @40  	dev_err(_dsp->dev, "DSP%d: " fmt, _dsp->num, ##__VA_ARGS__)
2159ad93 Mark Brown          2012-10-11  41  #define adsp_warn(_dsp, fmt, ...) \
2159ad93 Mark Brown          2012-10-11  42  	dev_warn(_dsp->dev, "DSP%d: " fmt, _dsp->num, ##__VA_ARGS__)
2159ad93 Mark Brown          2012-10-11  43  #define adsp_info(_dsp, fmt, ...) \
2159ad93 Mark Brown          2012-10-11  44  	dev_info(_dsp->dev, "DSP%d: " fmt, _dsp->num, ##__VA_ARGS__)
2159ad93 Mark Brown          2012-10-11  45  #define adsp_dbg(_dsp, fmt, ...) \
2159ad93 Mark Brown          2012-10-11  46  	dev_dbg(_dsp->dev, "DSP%d: " fmt, _dsp->num, ##__VA_ARGS__)
2159ad93 Mark Brown          2012-10-11  47  
2159ad93 Mark Brown          2012-10-11  48  #define ADSP1_CONTROL_1                   0x00

:::::: The code at line 40 was first introduced by commit
:::::: 2159ad936b7e7a8b26c99cf5b4476cfbb8c13e22 ASoC: adsp: Add ADSP base support

:::::: TO: Mark Brown <broonie@...nsource.wolfsonmicro.com>
:::::: CC: Mark Brown <broonie@...nsource.wolfsonmicro.com>

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

Download attachment ".config.gz" of type "application/gzip" (47887 bytes)