diff options
| author | Arnd Bergmann <arnd@arndb.de> | 2017-02-02 12:51:28 -0200 |
|---|---|---|
| committer | Mauro Carvalho Chehab <mchehab@s-opensource.com> | 2017-02-08 11:46:07 -0200 |
| commit | 69d3973af1acd4c0989ec8218c05f12d303cd7cf (patch) | |
| tree | 6008a0892b8b94a208621995d27edd18526f53ed /drivers/tc/tc.c | |
| parent | 005145378c9ad7575a01b6ce1ba118fb427f583a (diff) | |
| download | linux-69d3973af1acd4c0989ec8218c05f12d303cd7cf.tar.gz linux-69d3973af1acd4c0989ec8218c05f12d303cd7cf.tar.bz2 linux-69d3973af1acd4c0989ec8218c05f12d303cd7cf.zip | |
[media] ttpci: address stringop overflow warning
gcc-7.0.1 warns about old code in ttpci:
In file included from drivers/media/pci/ttpci/av7110.c:63:0:
In function 'irdebi.isra.2',
inlined from 'start_debi_dma' at drivers/media/pci/ttpci/av7110.c:376:3,
inlined from 'gpioirq' at drivers/media/pci/ttpci/av7110.c:659:3:
drivers/media/pci/ttpci/av7110_hw.h:406:3: warning: 'memcpy': specified size between 18446744071562067968 and 18446744073709551615 exceeds maximum object size 9223372036854775807 [-Wstringop-overflow=]
memcpy(av7110->debi_virt, (char *) &res, count);
In function 'irdebi.isra.2',
inlined from 'start_debi_dma' at drivers/media/pci/ttpci/av7110.c:376:3,
inlined from 'gpioirq' at drivers/media/pci/ttpci/av7110.c:668:3:
drivers/media/pci/ttpci/av7110_hw.h:406:3: warning: 'memcpy': specified size between 18446744071562067968 and 18446744073709551615 exceeds maximum object size 9223372036854775807 [-Wstringop-overflow=]
memcpy(av7110->debi_virt, (char *) &res, count);
Apparently, 'count' can be negative here, which will then get turned
into a giant size argument for memcpy. Changing the sizes to 'unsigned
int' instead seems safe as we already check for maximum sizes, and it
also simplifies the code a bit.
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Signed-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>
Diffstat (limited to 'drivers/tc/tc.c')
0 files changed, 0 insertions, 0 deletions