Bug 630098 - fix coverity Defect Type: Code maintainability issues
Summary: fix coverity Defect Type: Code maintainability issues
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: 389
Classification: Retired
Component: Directory Server
Version: 1.2.6
Hardware: All
OS: Linux
high
medium
Target Milestone: ---
Assignee: Nathan Kinder
QA Contact: Chandrasekar Kannan
URL:
Whiteboard:
Depends On:
Blocks: 639035
TreeView+ depends on / blocked
 
Reported: 2010-09-03 17:20 UTC by Nathan Kinder
Modified: 2015-01-04 23:43 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2011-05-17 14:08:10 UTC
Embargoed:


Attachments (Terms of Use)
Patch (1.14 KB, patch)
2010-09-03 17:39 UTC, Nathan Kinder
nhosoi: review+
nhosoi: review+
Details | Diff
Additional patch for coverity issue 15521 (1.17 KB, patch)
2010-09-03 18:25 UTC, Nathan Kinder
nhosoi: review+
nhosoi: review+
Details | Diff

Description Nathan Kinder 2010-09-03 17:20:30 UTC
fix coverity Defect Type: Code maintainability issues

Comment 1 Nathan Kinder 2010-09-03 17:39:25 UTC
Created attachment 442941 [details]
Patch

Comment 2 Nathan Kinder 2010-09-03 17:48:54 UTC
Pushed to master.  Thanks to Noriko for her review!

Counting objects: 13, done.
Delta compression using 2 threads.
Compressing objects: 100% (7/7), done.
Writing objects: 100% (7/7), 714 bytes, done.
Total 7 (delta 5), reused 0 (delta 0)
To ssh://git.fedorahosted.org/git/389/ds.git
   3c4185b..8eddca4  master -> master

Comment 3 Nathan Kinder 2010-09-03 18:25:08 UTC
Created attachment 442953 [details]
Additional patch for coverity issue 15521

Comment 4 Nathan Kinder 2010-09-03 18:33:18 UTC
Pushed additional patch to master.

Counting objects: 13, done.
Delta compression using 2 threads.
Compressing objects: 100% (7/7), done.
Writing objects: 100% (7/7), 722 bytes, done.
Total 7 (delta 5), reused 0 (delta 0)
To ssh://git.fedorahosted.org/git/389/ds.git
   8eddca4..644b553  master -> master

Comment 6 Amita Sharma 2011-05-13 07:00:34 UTC
Coverity Related, Can I request Dev to test this.


Note You need to log in before you can comment on or make changes to this bug.