allow net ads join accept new osServicePack parameter
authorNoel Power <noel.power@suse.com>
Thu, 2 Oct 2014 15:13:18 +0000 (16:13 +0100)
committerJeremy Allison <jra@samba.org>
Wed, 7 Jan 2015 23:18:05 +0000 (00:18 +0100)
commita5d383cbd56fdab958334c8e6a19a824941c11c1
treedbc5b33685964bcb19061e04d4d741908f47df2f
parenta13e29cc4345d85ab6fe4482119386b87e4e8673
allow net ads join accept new osServicePack parameter

osServicePack paramater allows the default behaviour ( which is to use
the samba version string as the operatingSystemServicePack attribute )
to be overridden
Additionally make sure if blank string is passed that it is treated
as attribute deletion. This is necessary as values for the os attributes
are eventually passed to ads_modlist_add if the value is "" then the
attempt to add this attribute fails in the underlying ldap
'ldap_modfiy_ext_s' function. In this case we need to pass NULL as the
value to force deletion of the ldap attribute

Signed-off-by: Noel Power <noel.power@suse.com>
Reviewed-by: David Disseldorp <ddiss@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
Autobuild-User(master): Jeremy Allison <jra@samba.org>
Autobuild-Date(master): Thu Jan  8 00:18:05 CET 2015 on sn-devel-104
source3/libnet/libnet_join.c
source3/librpc/idl/libnet_join.idl
source3/utils/net_ads.c