Cleanup: Fix scanner results
Reported by Alex (via GPGTools) | October 21st, 2011 @ 03:29 PM
Cleanup might be useful since we'll use this lib in all products. See attachment. Based on http://clang-analyzer.llvm.org :
- Dead store: 3
- Logic error: 2
- Memory leaks: 9
Comments and changes to this ticket
-
Alex (via GPGTools) October 21st, 2011 @ 03:32 PM
- no changes were found...
-
Mento October 21st, 2011 @ 05:51 PM
- State changed from “open” to “fixed”
-
Alex (via GPGTools) October 21st, 2011 @ 06:19 PM
- State changed from “fixed” to “open”
"make update check" says: 4 bugs found. Attached the report.
-
Mento October 21st, 2011 @ 06:44 PM
- State changed from “open” to “fixed”
I think i have fixed the bugs.
The scanner detects one false-positive bug. -
Alex (via GPGTools) October 22nd, 2011 @ 05:56 PM
- State changed from “fixed” to “waiting”
Could you have a look at my last changes - buildbot is now green.
https://github.com/GPGTools/Libmacgpg/commits/lion -
Mento October 22nd, 2011 @ 06:31 PM
I have reverted your changes at GPGException, because they can make problems if in self.errorCode in no error code.
-
Alex (via GPGTools) October 22nd, 2011 @ 10:07 PM
Ok, I might have a look at it again. I assume just another label/goto will be the easiest solution.
-- sent from my mobile phone,
please excuse my brevity -
-
Mento October 23rd, 2011 @ 11:10 AM
- State changed from “waiting” to “fixed”
Please Sign in or create a free account to add a new ticket.
With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.
Create new ticket
Create your profile
Help contribute to this project by taking a few moments to create your personal profile. Create your profile ยป
Alex (via GPGTools)
Luke Le
steve