[New-bugs-announce] [issue18521] [cppcheck] Full report

[Julien Nabet]

New submission from Julien Nabet:

Hello,

I retrieved Cpython sources today and runned cppcheck ("git updated" today) on the whole sources with enable=all
I attached the full report.
There are certainly false positive but some reports may help, eg:
[Python/getargs.c:379]: (style) Array index 'i' is used before limits check.
Indeed, here is the code:
    379             while (levels[i] > 0 && i < 32 && (int)(p-buf) < 220) {

[Modules/md5module.c:345] -> [Modules/md5module.c:342]: (style) Found duplicate branches for 'if' and 'else'
    342     if (Py_TYPE(self) == &MD5type) {
    343         if ( (newobj = newMD5object())==NULL)
    344             return NULL;
    345     } else {
    346         if ( (newobj = newMD5object())==NULL)
    347             return NULL;
    348     }

[Objects/iterobject.c:87]: (error) Uninitialized variable: seqsize
[Objects/setobject.c:549]: (error) Address of local auto-variable assigned to a function parameter
etc.

Hope it helps.

Julien

----------
files: cppcheck_reports.txt
messages: 193442
nosy: serval2412
priority: normal
severity: normal
status: open
title: [cppcheck] Full report
versions: Python 3.5
Added file: http://bugs.python.org/file30995/cppcheck_reports.txt

_______________________________________
Python tracker <report at bugs.python.org>
<http://bugs.python.org/issue18521>
_______________________________________