Ticket #1953 (reopened defect) — at Version 7
some cppcheck cleanups
| Reported by: | slyfox | Owned by: | slyfox |
|---|---|---|---|
| Priority: | minor | Milestone: | 4.7.5 |
| Component: | mc-core | Version: | master |
| Keywords: | cppcheck | Cc: | |
| Blocked By: | Blocking: | ||
| Branch state: | no branch | Votes for changeset: | commited-master commited-stable |
Description (last modified by slyfox) (diff)
http://apps.sourceforge.net/trac/cppcheck/
Issuing
mc $ cppcheck -a --force .
among many reports we get interesting hits
[./vfs/ftpfs.c:682]: (possible error) Memory leak: host [./src/util.c:1435]: (error) Resource leak: f
---
Merged as changeset:2c42692c7e0b3e98a694d5988d8c918289a03d64
$ git log 2c42692c7^..2c42692c7
Change History
comment:1 Changed 15 years ago by slyfox
- Status changed from new to accepted
- severity changed from no branch to on review
comment:3 Changed 15 years ago by andrew_b
- Votes for changeset changed from iNode to iNode andrew_b
- severity changed from on review to approved
comment:4 Changed 15 years ago by slyfox
- Status changed from accepted to testing
- Resolution set to fixed
- Description modified (diff)
comment:6 Changed 15 years ago by slavazanko
- Status changed from testing to closed
- Votes for changeset changed from iNode andrew_b to commited-master commited-stable
Cherry-picked and rebased into 4.7.0-stable: c87b4116bda53aded5d99006b454c909ed4ba581
comment:7 Changed 15 years ago by slyfox
- Status changed from closed to reopened
- Resolution fixed deleted
- Description modified (diff)
Reseted master and reapplied with fixed typo in label:
new merge tip: changeset:2c42692c7e0b3e98a694d5988d8c918289a03d64
Sorry for the shit.
Note: See
TracTickets for help on using
tickets.
Pushed as
branch:1953_cppcheck_resleaks (2 commits)
tip changeset:fca17611e5d0c38b2898ef68828c0e380fe7ab00
Please review.