Attachment 89921 Details for Bug 72088 – cppcheck report

cppcheck report

cppcheck_reports.txt (text/plain), 10.64 KB, created by Julien Nabet on 2013-11-27 22:41:34 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Julien Nabet

Created: 2013-11-27 22:41:34 UTC

Size: 10.64 KB

patch

obsolete

>[src/via_3d.c:260] -> [src/via_3d.c:261]: (warning) Possible null pointer dereference: v3d - otherwise it is redundant to check it against null.
>[src/via_3d.c:281]: (style) The scope of the variable 'scalex' can be reduced.
>[src/via_3d.c:281]: (style) The scope of the variable 'scaley' can be reduced.
>[src/via_3d.c:283]: (style) The scope of the variable 'vTex' can be reduced.
>[src/via_3d.c:546]: (style) The scope of the variable 'op' can be reduced.
>[src/via_3d.c:549]: (style) The scope of the variable 'format' can be reduced.
>[src/via_bandwidth.c:485] -> [src/via_bandwidth.c:483]: (style) Found duplicate branches for 'if' and 'else'.
>[src/via_bandwidth.c:502] -> [src/via_bandwidth.c:500]: (style) Found duplicate branches for 'if' and 'else'.
>[src/via_ch7xxx.c:528]: (style) The scope of the variable 'i' can be reduced.
>[src/via_display.c:1202] -> [src/via_display.c:1203]: (performance) Buffer 'dst' is being written before its old content has been used.
>[src/via_display.c:1004]: (style) The scope of the variable 'SR1A' can be reduced.
>[src/via_display.c:1004]: (style) The scope of the variable 'SR1B' can be reduced.
>[src/via_display.c:1004]: (style) The scope of the variable 'CR67' can be reduced.
>[src/via_display.c:1004]: (style) The scope of the variable 'CR6A' can be reduced.
>[src/via_display.c:1415]: (style) The scope of the variable 'SR1A' can be reduced.
>[src/via_display.c:1415]: (style) The scope of the variable 'SR1B' can be reduced.
>[src/via_display.c:1415]: (style) The scope of the variable 'CR67' can be reduced.
>[src/via_display.c:1415]: (style) The scope of the variable 'CR6A' can be reduced.
>[src/via_display.c:1030]: (style) Variable 'SR1A' is assigned a value that is never used.
>[src/via_display.c:1031]: (style) Variable 'SR1B' is assigned a value that is never used.
>[src/via_display.c:1032]: (style) Variable 'CR67' is assigned a value that is never used.
>[src/via_display.c:1033]: (style) Variable 'CR6A' is assigned a value that is never used.
>[src/via_dri.c:377]: (style) The scope of the variable 'i' can be reduced.
>[src/via_dri.c:377]: (style) The scope of the variable 'db' can be reduced.
>[src/via_dri.c:377]: (style) The scope of the variable 'stencil' can be reduced.
>[src/via_dri.c:377]: (style) The scope of the variable 'accum' can be reduced.
>[src/via_dri.c:860]: (style) The scope of the variable 'curSize' can be reduced.
>[src/via_dri.c:902]: (style) The scope of the variable 'err' can be reduced.
>[src/via_dri.c:71]: (style) struct or union member 'ViaDRMVersion::major' is never used.
>[src/via_dri.c:72]: (style) struct or union member 'ViaDRMVersion::minor' is never used.
>[src/via_dri.c:73]: (style) struct or union member 'ViaDRMVersion::patchlevel' is never used.
>[src/via_dri.c:481]: (error) Memory leak: pVIAConfigPtrs
>[src/via_dri.c:638]: (style) The scope of the variable 'pVIADRI' can be reduced.
>[src/via_driver.c:516]: (style) The scope of the variable 'i' can be reduced.
>[src/via_driver.c:60]: (style) struct or union member 'ViaDRMVersion::major' is never used.
>[src/via_driver.c:61]: (style) struct or union member 'ViaDRMVersion::minor' is never used.
>[src/via_driver.c:62]: (style) struct or union member 'ViaDRMVersion::patchlevel' is never used.
>[src/via_driver.c:1107]: (error) Memory pointed to by 'pEnt' is freed twice.
>[src/via_driver.c:901]: (error) Uninitialized variable: old_width
>[src/via_driver.c:900]: (error) Uninitialized variable: old_height
>[src/via_driver.c:902]: (error) Uninitialized variable: old_dwidth
>[src/via_driver.c:487]: (style) Variable 'entity' is assigned a value that is never used.
>[src/via_exa.c:222] -> [src/via_exa.c:220]: (warning) Possible null pointer dereference: cb - otherwise it is redundant to check it against null.
>[src/via_exa.c:939]: (style) The scope of the variable 'size' can be reduced.
>[src/via_exa.c:798] -> [src/via_exa.c:807]: (performance) Variable 'nPOTSupported' is reassigned a value before the old one has been used.
>[src/via_exa.c:679]: (style) The scope of the variable 'bufH' can be reduced.
>[src/via_exa.c:391]: (style) The scope of the variable 'size' can be reduced.
>[src/via_exa_h2.c:78]: (style) The scope of the variable 'i' can be reduced.
>[src/via_exa_h2.c:230]: (style) Variable 'cb' is assigned a value that is never used.
>[src/via_exa_h2.c:373]: (style) The scope of the variable 'offs' can be reduced.
>[src/via_exa_h6.c:78]: (style) The scope of the variable 'i' can be reduced.
>[src/via_exa_h6.c:231]: (style) Variable 'cb' is assigned a value that is never used.
>[src/via_exa_h6.c:386]: (style) The scope of the variable 'offs' can be reduced.
>[src/via_kms.c:286] -> [src/via_kms.c:287]: (performance) Buffer 'ptr' is being written before its old content has been used.
>[src/via_lvds.c:639]: (style) The scope of the variable 'PanelSizeString' can be reduced.
>[src/via_lvds.c:894]: (style) The scope of the variable 'resIdx' can be reduced.
>[src/via_lvds.c:894]: (style) The scope of the variable 'port' can be reduced.
>[src/via_lvds.c:894]: (style) The scope of the variable 'offset' can be reduced.
>[src/via_lvds.c:894]: (style) The scope of the variable 'data' can be reduced.
>[src/via_lvds.c:894]: (style) The scope of the variable 'j' can be reduced.
>[src/via_memmgr.c:110]: (style) The scope of the variable 'ret' can be reduced.
>[src/via_outputs.c:1098]: (style) Variable 'fref' is assigned a value that is never used.
>[src/via_ums.c:684]: (style) The scope of the variable 'bMemSize' can be reduced.
>[src/via_ums.c:223]: (style) The scope of the variable 'err' can be reduced.
>[src/via_vgahw.c:139]: (style) The scope of the variable 'i' can be reduced.
>[src/via_xv.c:316]: (style) The scope of the variable 'needBandWidth' can be reduced.
>[src/via_xv.c:316]: (style) The scope of the variable 'totalBandWidth' can be reduced.
>[src/via_xv.c:680]: (style) The scope of the variable 'numPorts' can be reduced.
>[src/via_xv.c:1025]: (style) The scope of the variable 'size' can be reduced.
>[src/via_xv.c:1177]: (style) The scope of the variable 'retCode' can be reduced.
>[src/via_xv.c:1204]: (style) The scope of the variable 'dstPitch' can be reduced.
>[src/via_xv.c:1539]: (style) The scope of the variable 'x' can be reduced.
>[src/via_xv.c:802]: (style) Variable 'attr' is not assigned a value.
>[src/via_xv.c:802]: (style) Variable 'avalue' is not assigned a value.
>[src/via_xv.c:1100]: (style) Variable 'lumaSync' is assigned a value that is never used.
>[src/via_xv_overlay.c:147]: (style) The scope of the variable 'count' can be reduced.
>[src/via_xv_overlay.c:465]: (style) The scope of the variable 'srcTopOffset' can be reduced.
>[src/via_xv_overlay.c:466]: (style) The scope of the variable 'srcLeftOffset' can be reduced.
>[src/via_xv_overlay.c:468]: (style) The scope of the variable 'n' can be reduced.
>[src/via_xv_overlay.c:564]: (style) The scope of the variable 'sw1' can be reduced.
>[src/via_xv_overlay.c:564]: (style) The scope of the variable 'd' can be reduced.
>[src/via_xv_overlay.c:678]: (style) The scope of the variable 'sh1' can be reduced.
>[src/via_xv_overlay.c:678]: (style) The scope of the variable 'd' can be reduced.
>[src/via_xv_overlay.c:1176]: (style) The scope of the variable 'addr' can be reduced.
>[src/via_xv_overlay.c:1808]: (style) The scope of the variable 'hqvSrcWidth' can be reduced.
>[src/via_xv_overlay.c:1808]: (style) The scope of the variable 'hqvDstWidth' can be reduced.
>[src/via_xv_overlay.c:1810]: (style) The scope of the variable 'tmp' can be reduced.
>[src/via_xv_overlay.c:1812]: (style) The scope of the variable 'i' can be reduced.
>[src/via_xv_overlay.c:1214]: (portability) 'buf' is of type 'void *'. When using void pointers in calculations, the behaviour is undefined.
>[src/via_xv_overlay.c:1145]: (style) Variable 'width' is assigned a value that is never used.
>[src/via_xvmc.c:686]: (style) The scope of the variable 'sAPriv' can be reduced.
>[src/via_xvmc.c:822]: (style) The scope of the variable 'pPriv' can be reduced.
>[src/via_xvmc.c:823]: (style) The scope of the variable 'vx' can be reduced.
>[src/via_xvmc.c:866]: (style) The scope of the variable 'i' can be reduced.
>[src/via_xvmc.c:890]: (style) The scope of the variable 'i' can be reduced.
>[src/via_xvmc.c:912]: (style) The scope of the variable 'ret' can be reduced.
>[src/via_xvmc.c:605]: (portability) 'buf' is of type 'void *'. When using void pointers in calculations, the behaviour is undefined.
>[src/via_xvmc.c:606]: (portability) 'buf' is of type 'void *'. When using void pointers in calculations, the behaviour is undefined.
>[src/via_xvmc.c:694]: (style) Variable 'sAPriv' is assigned a value that is never used.
>[src/xvmc/viaLowLevel.c:273]: (style) The scope of the variable 'ret' can be reduced.
>[src/xvmc/viaLowLevelPro.c:505]: (style) The scope of the variable 'i' can be reduced.
>[src/xvmc/viaLowLevelPro.c:611]: (style) The scope of the variable 'ret' can be reduced.
>[src/xvmc/viaLowLevelPro.c:880]: (style) The scope of the variable 'i' can be reduced.
>[src/xvmc/viaLowLevelPro.c:1456]: (style) The scope of the variable 'ret' can be reduced.
>[src/xvmc/viaLowLevelPro.c:1468]: (style) Variable 'ret' is assigned a value that is never used.
>[src/xvmc/viaXvMC.c:160]: (style) The scope of the variable 'curAD' can be reduced.
>[src/xvmc/viaXvMC.c:1070]: (style) The scope of the variable 'i' can be reduced.
>[src/xvmc/viaXvMC.c:1414]: (style) The scope of the variable 'dAddr' can be reduced.
>[src/xvmc/viaXvMC.c:1414]: (style) The scope of the variable 'sAddr' can be reduced.
>[src/xvmc/viaXvMC.c:1688]: (style) The scope of the variable 'pViaXvMC' can be reduced.
>[src/xvmc/viaXvMC.c:1689]: (style) The scope of the variable 'sAPriv' can be reduced.
>[src/xvmc/viaXvMC.c:1743]: (style) The scope of the variable 'sAPriv' can be reduced.
>[src/xvmc/viaXvMC.c:1744]: (style) The scope of the variable 'i' can be reduced.
>[src/xvmc/viaXvMC.c:55]: (style) struct or union member 'ViaDRMVersion::patchlevel' is never used.
>[src/xvmc/viaXvMC.c:1098]: (style) Variable 'timeStamp' is assigned a value that is never used.
>[src/xvmc/viaXvMC.c:1638]: (style) Variable 'pViaSubPic' is assigned a value that is never used.
>[src/xvmc/xf86dri.c:502]: (style) Variable 'total_rects' is assigned a value that is never used.
>[tools/registers.c:901]: (warning) %d in format string (no. 2) requires 'int' but the argument type is 'unsigned int'.
>[tools/registers.c:901]: (warning) %d in format string (no. 3) requires 'int' but the argument type is 'unsigned int'.
>[tools/registers.c:901]: (warning) %d in format string (no. 4) requires 'int' but the argument type is 'unsigned int'.
>[tools/registers.c:901]: (warning) %d in format string (no. 5) requires 'int' but the argument type is 'unsigned int'.
>[tools/registers.c:964]: (warning) %u in format string (no. 2) requires 'unsigned int' but the argument type is 'int'.
>[tools/registers.c:1033]: (style) Variable 'sl_size_mb' is assigned a value that is never used.

Actions: View

Attachments on bug 72088: 89921 | 90136 | 90137 | 90809 | 90810 | 90811 | 90813

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.