Bugzilla (bugzilla.redhat.com) will be under maintenance for infrastructure upgrades and will not be available on July 31st between 12:30 AM - 05:30 AM UTC. We appreciate your understanding and patience. You can follow status.redhat.com for details.
Bug 623507 - fix coverity Defect Type: Incorrect expression issues
Summary: fix coverity Defect Type: Incorrect expression issues
Alias: None
Product: 389
Classification: Retired
Component: Directory Server
Version: 1.2.6
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: Noriko Hosoi
QA Contact: Chandrasekar Kannan
Depends On:
Blocks: 389_1.2.7 639035
TreeView+ depends on / blocked
Reported: 2010-08-12 00:19 UTC by Noriko Hosoi
Modified: 2015-01-04 23:43 UTC (History)
1 user (show)

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

Attachments (Terms of Use)
git patch file (master) (1.14 KB, patch)
2010-08-12 00:55 UTC, Noriko Hosoi
nkinder: review+
Details | Diff
git patch file (master) (951 bytes, patch)
2010-08-12 01:02 UTC, Noriko Hosoi
nkinder: review+
Details | Diff

Description Noriko Hosoi 2010-08-12 00:19:14 UTC
Description of problem:
fix coverity Defect Type: Incorrect expression issues

Comment 1 Noriko Hosoi 2010-08-12 00:55:34 UTC
Created attachment 438320 [details]
git patch file (master)


be_list is never NULL so there is no need to check for NULL value.

Comment 2 Noriko Hosoi 2010-08-12 01:02:57 UTC
Created attachment 438321 [details]
git patch file (master)


readonly_attributes is never NULL so it's not necessary to compare
with NULL.

Comment 3 Noriko Hosoi 2010-08-20 22:07:07 UTC
Reviewed by Nathan (Thank you!!)

Pushed to master:

$ git merge coverity
Updating d5b1593..d7c4d9b
 ldap/servers/slapd/opshared.c |    2 +-
 ldap/servers/slapd/rootdse.c  |    4 ----
 2 files changed, 1 insertions(+), 5 deletions(-)

$ git push
Counting objects: 18, done.
Delta compression using up to 4 threads.
Compressing objects: 100% (12/12), done.
Writing objects: 100% (12/12), 1.18 KiB, done.
Total 12 (delta 9), reused 0 (delta 0)
To ssh://git.fedorahosted.org/git/389/ds.git
   d5b1593..d7c4d9b  master -> master

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