[Openchrome-devel] [Bug 72088] New: cppcheck report

[bugzilla-daemon at freedesktop.org]

https://bugs.freedesktop.org/show_bug.cgi?id=72088

          Priority: medium
            Bug ID: 72088
          Assignee: openchrome-devel at lists.freedesktop.org
           Summary: cppcheck report
          Severity: normal
    Classification: Unclassified
                OS: All
          Reporter: serval2412 at yahoo.fr
          Hardware: Other
            Status: NEW
           Version: git
         Component: Driver/openchrome
           Product: xorg

Created attachment 89921
  --> https://bugs.freedesktop.org/attachment.cgi?id=89921&action=edit
cppcheck report

I runned cppcheck git updated today on OpenChrome git updated today.

There are certainly false positives since cppcheck is a static analyzer for C++
(I think about "scope of <var> can be reduced) but for the rest, it could help.
eg:
[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_driver.c:1107]: (error) Memory pointed to by 'pEnt' is freed twice.
[src/via_dri.c:481]: (error) Memory leak: pVIAConfigPtrs

[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

I attached the full report (I used --enable=all for running cppcheck).

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/openchrome-devel/attachments/20131127/c0663615/attachment.html>