69d3973af1
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> |
||
---|---|---|
.. | ||
b2c2 | ||
bt8xx | ||
cobalt | ||
cx18 | ||
cx88 | ||
cx23885 | ||
cx25821 | ||
ddbridge | ||
dm1105 | ||
dt3155 | ||
ivtv | ||
mantis | ||
meye | ||
netup_unidvb | ||
ngene | ||
pluto2 | ||
pt1 | ||
pt3 | ||
saa7134 | ||
saa7146 | ||
saa7164 | ||
smipcie | ||
solo6x10 | ||
sta2x11 | ||
ttpci | ||
tw68 | ||
tw686x | ||
tw5864 | ||
zoran | ||
Kconfig | ||
Makefile |