Bug 719069 - clean up compiler warnings in 389-ds-base 1.2.9
clean up compiler warnings in 389-ds-base 1.2.9
Status: CLOSED CURRENTRELEASE
Product: 389
Classification: Community
Component: Directory Server (Show other bugs)
1.2.9
Unspecified Unspecified
unspecified Severity unspecified
: ---
: ---
Assigned To: Noriko Hosoi
Viktor Ashirov
: VerifiedUpstream
Depends On:
Blocks: 389_1.2.9
  Show dependency treegraph
 
Reported: 2011-07-05 12:39 EDT by Noriko Hosoi
Modified: 2015-12-07 11:38 EST (History)
3 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2015-12-07 11:38:52 EST
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:


Attachments (Terms of Use)
git patch file (master) (7.27 KB, patch)
2011-07-05 12:54 EDT, Noriko Hosoi
nkinder: review+
Details | Diff

  None (edit)
Description Noriko Hosoi 2011-07-05 12:39:47 EDT
Description of problem:
dap/servers/slapd/entry.c:528: warning: passing argument 2 of '_entry_set_tombstone_rdn' discards qualifiers from pointer target type
ldap/servers/slapd/entry.c:1240: warning: passing argument 2 of '_entry_set_tombstone_rdn' discards qualifiers from pointer target type
ldap/servers/slapd/mapping_tree.c:1003: warning: 'mapping_tree_node_validate' defined but not used
ldap/servers/plugins/automember/automember.c:1709: warning: label 'bail' defined but not used
ldap/servers/plugins/automember/automember.c:1692: warning: unused variable 'smods'
ldap/servers/plugins/automember/automember.c:1752: warning: implicit declaration of function 'slapi_entry_attr_hasvalue'
ldap/servers/plugins/mep/mep.c:2108: warning: implicit declaration of function 'slapi_entry_attr_hasvalue'
ldap/servers/plugins/replication/windows_protocol_util.c:2169: warning: passing argument 2 of 'slapi_entry_apply_mod' discards qualifiers from pointer target type
Comment 1 Noriko Hosoi 2011-07-05 12:54:58 EDT
Created attachment 511349 [details]
git patch file (master)

Description:
slapd/entry.c
  -- added const qualifier to the 2nd arg of _entry_set_tombstone_rdn.
slapd/mapping_tree.c
  -- commented out mapping_tree_node_validate.
plugins/automember/automember.c
  -- removed a unused variable smods.
  -- modified to use label "bail".
  -- replaced deprecated slapi_entry_attr_hasvalue with
     slapi_entry_attr_has_syntax_value.
plugins/mep/mep.c
  -- replaced deprecated slapi_entry_attr_hasvalue with
     slapi_entry_attr_has_syntax_value.
Comment 2 Noriko Hosoi 2011-07-05 13:40:28 EDT
Reviewed by Nathan (Thank you!!!)

Pushed to master.

$ git merge work
Updating f069f4b..dfcc435
Fast-forward
 ldap/servers/plugins/automember/automember.c       |   14 +++++++----
 ldap/servers/plugins/mep/mep.c                     |   24 ++++++++++++-------
 .../plugins/replication/windows_protocol_util.c    |    2 +-
 ldap/servers/slapd/entry.c                         |    4 +-
 ldap/servers/slapd/mapping_tree.c                  |    3 ++
 5 files changed, 30 insertions(+), 17 deletions(-)

$ git push
Counting objects: 27, done.
Delta compression using up to 2 threads.
Compressing objects: 100% (14/14), done.
Writing objects: 100% (14/14), 1.79 KiB, done.
Total 14 (delta 10), reused 0 (delta 0)
To ssh://git.fedorahosted.org/git/389/ds.git
   f069f4b..dfcc435  master -> master

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