Fix a comment.

Michael

Rename libnet_smbconf_reg_createkey_internal to libnet_smbconf_reg_create_service_key.

Michael

Add libnet_conf API function libnet_smbconf_create_share().

And make libnet_smbconf_setparm() return error if the share
does not already exist. Adapt net_conf_addshare to this new
situation.

Michael

Remove unused vars.

Guenther

Trivial simplification

... things you come across when you review code

Add comments

Jeremy was able to use this correctly without comments, so the code can't be
*that* bad :-)

Fix for bug #5163 from Laurent Pinchart <pinchart@skynet.be>
Failure to change password in ldap is mapped to NT_STATUS_UNSUCCESSFUL unconditionally.
Jeremy.

Convert the little caches in util_sock.c to use
the singleton memcache. Vl please check (passes
make valgrindtest).
Jeremy.

Don't leak: Use a temporary context for the admin token and free it.

Michael

Use libnet_smbconf_reg_open_path() in libnet_smbconf_drop().

Replaces creation of token and direct use of reg_open_path.

Michael

Abstract opening of registry path out of libnet_smbconf_reg_open_service_key().

Creates new function libnet_smbconf_reg_open_path().
Use libnet_smbconf_reg_open_path() directly in libnet_smbconf_reg_open_basekey().
Return error in libnet_smbconf_reg_open_service_key() when NULL
servicename is given.

Michael

Add debug output in error path.

Michael

Fix setting of error code in error path.

Michael

Use a better parameter name.

Michael

Rename libnet_smbconf_reg_open_path() to libnet_smbconf_reg_open_service_key().

Michael

Hey, it is 2008 now.

:-)

Michael

Choose a more apropriate parameter name.

Michael

Rename libnet_smbconf_reg_open_basepath() to libnet_smbconf_reg_open_basekey().

Michael

Fix some C++ warnings (patch was Volker's) - implicit case from void* to char*

Merge branch 'work/v3-2-test' into v3-2-test

use C99 bool return types (true & false).

Make sure that wbcLookupSid() and wbcLookupRids() use talloc()'d memory.

Follows existing convention that all returned memory should be freed with
wbcFreeMemory() and not directly with free().  Noticed by Volker.  Txs.

Attempt to fix bug #3617. Mix of patches from Volker and
myself. Use standard dlinklist macros.
Jeremy.

Use the proper boolean constants.

Michael

Fix libnetapi (resolve dependency on libwbclient).

Guenther

Fix libsmbclient (resolve dependency on libwbclient).

Guenther

Make libsmbsharemodes.so work again (fix unresolved tdb symbols).

Guenther

Convert call_trans2open to create_file()

Convert reply_mknew to create_file

Convert reply_open_and_X to create_file

Convert reply_open to create_file

Add test_shlibs to Makefile.

Guenther

Happy new year !

Guenther

Use NULL instead of 0

"struct security_descriptor" has pointers, not integers inside

Make the last two helper functions in libnet_conf.c static.

Now the registry backend is completely hidden from the API.

Michael

Make grouping in if statement more explicit.

Michael

Fix a permissions error in reg_deletekey_recursive().

Michael

Change net_conf_import() to only use libnet_conf API functions.

More precisely, only import_process_service() is changed.
This removes all references to registry code from net_conf.c.

net_conf_import() is currently -- like net_conf_addshare() -- also
considered a high-level add-on, not an API function.

Michael

Rewrite net_conf_addshare() to only use libnet_conf API functions.

Also exit on error if the share already exists.

net_conf_addshare() is considered a high level utility function.
It should not be an libnet_conf API function in itself since it is
kind of arbitrary.

Michael

Rename libnet_smbconf_key_exists() to libnet_smbconf_share_exists()

and move it to the api section of libnet_conf.c

Michael

Remove an unused variable.

Michael

Remove an unused variable.

Michael

Remove all d_fprintf-s from libnet_conf.c

Replacing them buy DEBUG statements and filling in
d_fprintfs in callers in net_conf.c.

Michael

As the encryption is stream based there's no reason
oplock breaks can't be encrypted. If we have multiple
contexts I should probably attach them to the connection
struct, but for now use the global context number.
Jeremy.

Fix error message.
Jeremy.

Added -e, --encrypt option to smbclient that immediately
forces encrypted smb after initial connect. Will document
for 3.2 official release.
Jeremy.

Allow encryption context setup on IPC$.
Jeremy.

Make pdb_tdb honour a private dir overridden in smb.conf.

One lp_private_dir() has to be used instead of get_dyn_PRIVATE_DIR()
to determine the location of the passdb.tdb.

I noticed this when running make test as a "normal user" from a
build, where I had done "make install" as root before, and so
the passdb.tdb could not be accessed during the startup phase
"CREATE TEST ENVIRONMENT IN ./st ..." in selftest.sh.

Michael

Make libnet_smbconf_reg_open_path() static.

Michael

Make libnet_smbconf_reg_open_basepath() static.

Michael

Remove list_values() from net_conf.c - it is not needed any more.

Also make libnet.c:libnet_smbconf_format_registry_value() static.
(There are nor more external callers.)

Michael

Use libnet_smbconf_get_config() in net_conf_list().

This leaves only output logic in net_conf_list().

Michael

Add a function libnet_smbconf_get_config() to libnet_conf.c

This gets the whole config as a set of lists (of share names
and corresponding lists of parameter names and values). The function
is an aggregate of libnet_smbconf_get_share_names() and
libnet_smbconf_getshare().

Michael

Include libnet/libnet.h in libnet_conf.c to have prototypes available.

Michael

Use correct size value for linearize call.
Jeremy.

Remove tiny code duplication

ndr_size_security_descriptor does the same as sec_desc_size

Make [un]marshall_sec_desc use librpc/ndr

Make use of [un]marshall_sec_desc

Remove tiny code duplication

sid_size did the same as ndr_size_dom_sid

Fix a panic

get_root_nt_token asks for "struct nt_user_token". talloc_get_type is not smart
enough to see that this is the same as NT_USER_TOKEN... :-)

Apply some const

Return NULL (instead of unchanged) for no shares/parameters defined.

Michael

Dont return count - 1 but count from libnet_smbconf_reg_get_values().

Michael

Make sure libnet_smbconf_get_share_names() always lists "global" first.

And don't return count-1 but count.

Michael

Move talloc-appending a string to an array to its own helper function

libnet_smbconf_add_string_to_array().

Michael

Fix uninitalized variables

Attempt to fix the AIX make test failures

For some reason AIX does not return the sender address in the AF_UNIX recvfrom.
So the faked netbios name lookup does not work with socket wrapper, nmbd can't
know where to send the reply. This patch works around this by telling smbclient
explicitly where to connect.

If there's any AIX experts listening -- how do I get the sender address from
AF_UNIX recvfrom?

Volker

Fix a const warning.

Michael

Rename libnet_smbconf_getshares() to libnet_smbconf_get_share_names().

Michael

Add a comment header for libnet_smbconf_getshares().

Michael

Move functionality of net_conf_listshares() to libnet_conf.c

into new function libnet_smbconf_getshares().

Michael

Move functionality of net_conf_showshare() to libnet_conf.c

The functionality is moved to a new function libnet_smbconf_getshare().
This returns the parameters of the given share as two lists: the list
of parameter names and the list of matching (formatted) parameter values.

The retrieval and formatting is done in a new internal helper function
libnet_smbconf_reg_get_values() that is to become the replacement for
list_values() from net_conf.c once functionality of net_conf_list() has
been moved to libnet_conf, too.

Michael

passdb.tdb is located in the private directory

Jerry, as part of d6cdbfd87 the default location of passdb.tdb has changed from
the private directory to the state directory. I think because passdb.tdb holds
the password hashes, it is reasonable to keep this next to the smbpasswd file.

Please review and potentially push.

Thanks,

Volker

Make the main net_conf functions static in net_conf.c

Michael

Add a couple of comment headers to the main libnet_conf functions.

Michael

Add a comment header for libnet_smbconf_format_registry_value().

Michael

Handle NULL talloc context in libnet_smbconf_format_registry_value().

Maybe we should generate a new context instead of returning NULL?

Michael

Hide the registry backend from libnet_smbconf_getparm().

Return a formatted string of the value instead.

Michael

Rename format_value() to libnet_smbconf_format_registry_value().

Michael

Move format_value() to libnet_conf.c.

Michael

Convert get_root_nt_token to memcache

Remove a global

don't store cache_readonly in gencache

tdb won't allow us to write anyway

Remove static zeros

Remove a static array

This might be worth it, mangle_hash is hardly used these days

Convert csamuser to memcache

bin/vlp needs a dependency on @LIBWBCLIENT_SHARED@

This failed to link if you compile with -j.

Improve error messages of 'net rpc user [add|delete]' commands.

Karolin

Ensure we turn off POSIX large read/write if encryption
is mandatory or signing is on.
Jeremy.

Add the capability to set "smb encrypt = required"
on a share (or global) and have the server reply with
ACCESS_DENIED for all non-encrypted traffic (except
that used to query encryption requirements and set
encryption state).
Jeremy.

Fix setting the initial permission bits

This fixes a make test failure on Solaris. When creating a new file,
file_set_dosmode() called from open_file_ntcreate calculates a new permission
mask, very likely different from what had been calculated in
open_file_ntcreate. Further down we overwrote the newly calculated value with
SMB_FCHMOD_ACL, ignoring what file_set_dosmode had calculated.

Why did Linux not see this? fchmod_acl on a newly created file without acls
would not retrieve an acl at all, whereas under Solaris acl(2) returns
something even for files with just posix permissions returns something.

Jeremy, given that we have very similar code in 3.0.28 this might also explain
some of the bug reports that people have concerning ACLs on new files.

Volker

P.S: This one took a while to find...

Add "smb encrypt" parameter. Can be set to "no, yes, required".
Currently if set required this is not enforced. I'll be adding
that soon.
Jeremy.

Fix the non-gcc branch of "likely"

Remove a silly static

Fix the build

Wrap the DEBUG checks in a "unlikely"

On my Laptop with some limited netbench runs this gains about 1.5% of
performance. When looking at the assembler output I would suspect the biggest
gain is by the fact that with this in place the calls to the debug functions is
moved to the function end, out of the way of the normal code paths. valgrind
tests pending I would suspect this to be much more cache friendly.

Comments?

Volker

Add CIFS_UNIX_TRANSPORT_ENCRYPTION_CAP capability to
our reported caps.
Jeremy.

Encryption works better when you add the client decrypt code :-).
Jeremy.

Add SMB encryption. Still fixing client decrypt but
negotiation works.
Jeremy.

typos