Merge branch 'setxattr-dos-mode' into v3-2-test
(This used to be commit 4c08725a42a8babf0f078fb4b221a96f58deeb50)

Allow clearing all settable DOS mode bits.  A mode value of zero is ignored by
Windows.  If the requested mode is zero, we instead send the appropriate one
of 0x80 (NORMAL) or 0x10 (DIRECTORY).

Thanks Jeremy!

Derrell
(This used to be commit 54abf7d0e595e9cbeea115a40d4f7b995252a150)

Don't leak memory in error path.
Jeremy.
(This used to be commit 2df0cdaafdced798f81e30d34371aa1d8e963208)

Use strchr_m in seaching for '.' in the hostname to make sure we're mb safe.
Jeremy.
(This used to be commit 090061b73a1c086ff8a7797e1a63532eacd91148)

Fix Windows 2008 (Longhorn) join.

During 'net ads join' the cli->desthost is a hostname (e.g.
rupert.galaxy.site). Check if we have a hostname and use only the
first part, the machine name, of the string.
(This used to be commit 5f60ed4af680ba2811db8d9f8267348ce05f26d2)

Windows 2008 (Longhorn) auth2 flag fixes.

Interop fixes for AD specific flags. Original patch from Todd Stetcher.
(This used to be commit 5aadfcdaacd6f136eab9e107a88b8544e6d2105f)

Forward ported version of Matt Geddes <musicalcarrion@gmail.com>
patch for adding acct_flags to rpccli_samr_create_dom_user().
Jerry please test.
Jeremy.
(This used to be commit 7d94f97947b7edfcf3ec52f0125e4593d6d54c05)

read_socket_with_timeout has timeout=0 handling
(This used to be commit 7101026061c470ed962267b43ac0aa67cc761a64)

strtok -> strtok_r
(This used to be commit fd34ce437057bb34cdc37f4b066e424000d36789)

Fix panic: Don't free the logfilename in winbind_child_died().

The child struct is immediately reused, and this results
in a panic when child->logfilename == NULL.

Michael
(This used to be commit da131d089db98017632103aa9bbe38c98f7a3fc1)

Initialize _domain_list to NULL.

Just to be sure the "if (!_domain_list)" in domain_list() test always works.

Michael
(This used to be commit 1f49065d44dd7570d5a9928359751bd36f287952)

Fix get_trust_creds() to return always an upper-cased krb5 principal (this
fixes winbind krb5 session at least with heimdal).

Guenther
(This used to be commit 9cf3a98eacea2dd07f89245f147e002b3f49482e)

Remove two unused variables
(This used to be commit f12ac1ecedd6634d8ccc54d8fce8382514e3322f)

Fix tab_depth: it should not create an extra debug header.

In pstring removal 4ae4b23586, the behaviour of tab_depth was
changed to create an extra debug header (by using the DEBUGLVL
macro).

This extracts the debug level check from DEBUGLVL into
a macro CHECK_DEBUGLVL without the debug header creation
and uses this instead of DEBUGLVL in tab_depth.

Michael
(This used to be commit cbc7d921fa696e6c3c5197ad9f87442ba679df82)

Merge branch 'v3-2-test' of ssh://git.samba.org/data/git/samba into v3-2-test
(This used to be commit bc2973df8504850a40cb0a1172689dc0bdafa323)

Added PFIF notice to Samba 3.2 tree
(This used to be commit afccc9703c45815f94dd05805a060d71b7fe618a)

Get Samba version or capability information from Windows

On Jan 22 13:31, Volker Lendecke wrote:
> On Tue, Jan 22, 2008 at 11:33:17AM +0100, Corinna Vinschen wrote:
> > Right.  I changed samba_gitcommitdate from time_t to NTTIME and shortened
> > samba_version_string to 28 bytes.  New patch below.
>
> Ok, pushed with some modifications. You might want to review
> that.

Reviewed and tested.  Looks good, thank you!

Below you'll find a tiny patch to add the git commit date.  It seems
I simply missed its existence in version.h :(  Tested on Windows XP.

Thanks,
Corinna

* source/smbd/trans2.c (samba_extended_info_version): Fill out
samba_gitcommitdate member with GIT commit timestamp.
(This used to be commit a33d0797585b5cdd304ac1eb6966b9c7c799bab6)

replace svn references with git ones in Readme
(This used to be commit e6c92b62a4791fd5b530e6e52e4386e837bd7994)

Move samba_extended_info_version to smbd/trans2.c

This is right now only used there, and in version.c it gave linker errors
because some binaries (e.g. smbmnt) don't link in time.o
(This used to be commit 1f0eaaa5911f893c822465a26fe49ab65afb0730)

Avoid use of uninitialized memory
(This used to be commit 85123aacdb13e97c3f44aeded1c80e13af53d83d)

Get Samba version or capability information from Windows

On Jan 21 16:18, Danilo Almeida wrote:
> Corina wrote:
>
> > + time_t samba_gitcommitdate;
>
> And:
>
> > +   SIVAL(pdata,28,extended_info.samba_gitcommitdate);
> > +   memcpy(pdata+32,extended_info.samba_version_string,32);
>
> Note that you are dropping bits on a system w/64-bit time_t, and that this has the 2038 problem.

Right.  I changed samba_gitcommitdate from time_t to NTTIME and shortened
samba_version_string to 28 bytes.  New patch below.

Thanks,
Corinna
(This used to be commit 28aa1c199d3a22cda34afcaab49c0561eeb0abcb)

Hide streams from the EA API
(This used to be commit e9bb3d5067b74a29beb778f85687829778e42b5b)

libreplace: getpwent_r/getgrent_r on IRIX are similar to solaris but use size_t

metze
(This used to be commit 2f460915111066d79f5dc9b4ae4d003918d06852)

Fix a ton of IBM checker uninitialized variable warnings

SET_STAT_INVALID only sets nlink, not the other fields

We might consider to change SET_STAT_INVALID to always do ZERO_STRUCT
(This used to be commit 8cf8c5b2034fe093b5db7f069bc6be8d328399bf)

Fix get_ea_names_from_file for many EAs

Found by the IBM checker
(This used to be commit 8741a9b37496e78b28d59d844aaba12f269171e0)

Fix a segfault

Found by the IBM checker.

Jeremy, you changed this from 'dirname == 0' which does not make sense at all
in the old code to '!dirname == 0' which from my point of view makes even less
sense, because the precedence of ! is according to Harbison/Steele higher than
the precedence of ==. Please check that the code now actually does what it's
supposed to do.

Thanks!
(This used to be commit c8def5564739c4c754001f84fcfb77f1093b96c3)

Add another check for talloc failure to libnet_conf

Michael
(This used to be commit 196e4ce8c5dd5aab518aaa7d170eb1fb5d66bcd1)

Fix formatting of multi_sz registry values.

Don't print only the last component.

Michael
(This used to be commit 654e96208ec847e32797cbd2442ef9e73c014567)

Use talloc_stackframe() for temporary contexts throughout libnet_conf.c

Michael
(This used to be commit 4d734106b70b9b6029b537fe11f8b3c1aebd42cf)

Add a check for talloc failure. - Pointed out by Volker.

Michael
(This used to be commit bdc49b07cc6de36c9319254a131858c9a7f9dd53)

Fix two debug messages.

Michael
(This used to be commit 35e23368dd4240a3c907b4cee882f51119032527)

Attempt to fix the darwin build
(This used to be commit 5b05e09daf18eaea5e86dfd607c8070228f7571b)

util_sock: Don't return a pointer to freed memory.

Fix a bug in my bugfix. Thanks to vl for spotting that one.
(This used to be commit 24f68b90cca111256a7b03f7062cb57c2b08a0d6)

Add SMB_VFS_FS_CAPABILITIES

It turns out that this is a necessary operation, separate from statvfs. statvfs
can fail during tcon, so conn->fs_capabilities would never see that we support
streams on a particular share.

James, can you check that I got the darwin variant right? Thanks!
(This used to be commit 3ad798d803b3b023533bb48e6993885f22b96095)

Link smbd on AIX with -bexpfull instead of -bexpall

-bexpall only exports the names without leading _, so we don't get
_talloc_reallo_array for example.

Thanks a lot to Heinrich Mislik for the hint!
(This used to be commit 74a71f198866ba0a0217e58a6a47bdc858d5df68)

util_sock: Fix memcache bug in get_mydnsfullname.

get_mydnsfullname relied on memcache_add(); memcache_lookup() working.
When run from ntlm_auth, the global_cache variable in memcache is NULL, so
the add and lookup both fail. In that case, just return the result of the
getaddrinfo call.

Jeremy, please check.
(This used to be commit 1db41ff52565e9f336a22fb9ffd80d51677e023b)

libwbclient: use the same style in each switch case for the error mapping

metze
(This used to be commit 0b85f7173d0770f8d9a0bac9e43355eb45bc181f)

winbindd: remove useless strcpy

metze
(This used to be commit df08708fc1e8fc8e15b36db29faf35ae5ae64b65)

Rename lib/util_reg_smbconf.c to registry/reg_init_smbconf.c

This actually is a counterpart to reg_init_full, in that is does
open and initialize the registry too, but only registeres the backends
necessary to access the SMBCONF key.

Michael
(This used to be commit 01bda3ab359fb3868c1dc849044f613bf2bc563e)

Move reg_frontend_hilvl.c to reg_dispatcher.c

This is actually the dispatcher to the registered registry backends.

Michael
(This used to be commit 39d65d11f8eff0aff998d5bfed8480b0f00655bd)

Extend/fix comments.

Michael
(This used to be commit 4952417fca89f5d797c861cde9fe74050ae02e2f)

Extract regkey_open_internal() from reg_frontend.c to new reg_util_legacy.c

reg_openpath should be used instead of this function (along with the
reg_api interface). Last callers of this function are in services_db.c

Michael
(This used to be commit 0005b88d74fca1ea1410c9911d504b0a636a0472)

Rename reg_db.c to reg_backend_db.c

Michael
(This used to be commit c3f695d3b14ee06fc5d517ca094236e885f9e707)

Rename reg_shares.c to reg_backend_shares.c

Michael
(This used to be commit aaf33ae5b8bfaefd2342c9ce6363577ce7c0c4f1)

Rename reg_smbconf.c to reg_backend_smbconf.c

Michael
(This used to be commit b85e5ee45075211dd3ab85ac79c2d856187d5e3e)

Rename reg_printing.c to reg_backend_printing.c

Start making naming of source files more systematic
to facilitate understanding of the structures.

Michael
(This used to be commit b3c2fb17a4226559788a47fee968ef19b7fbb6b2)

Makefile.in: Group together registry related object variables.

Michael
(This used to be commit cf485273f61e3dcd41ab7af756cc82f32131bf08)

Makefile.in: extract registry backend objects to common variable.

Michael
(This used to be commit 562c3eb37e2ff2b385203be9d556daf37df7d20e)

reg_eventlog.o and reg_perfcount.o are no longer neede for reg_smbconf.

This is an effect of the removal of the dynamic registry overlay.
This is the beginning of a restructuring of registry linkage in Makefile.in.

Michael
(This used to be commit 7453e78a8e63aaaff57c1779d6b7cf0ec8fc263f)

Fix some "set but never used" warnings
(This used to be commit 4a6dadc5178f4861e9c032321939db3b639734b5)

Free case_state earlier

Found by a "set but never used" warning. Thanks to talloc_tos() this was not
really a bug, but this way the code becomes much clearer.
(This used to be commit b326f11dc39a8ce20d957aac976be0cf7108ba9d)

Activate SMB_QUERY_FILE_STREAM_INFO

gcolley was right, my nt4 does not blue screen with it
(This used to be commit 9f2b5dc1865ca109e1e34c91548479ad9b26f450)

Active RAW-STREAMS in make test
(This used to be commit 63ee2ef775b48fcaecccf4dc7ef3601ceb8053c5)

Fix valgrind errors
(This used to be commit d7e6ec2258350c564053371361c8f1d7d0f775b1)

NT_STATUS_OBJECT_NAME_NOT_FOUND also means "no streams around :-)"
(This used to be commit 96b9a7b3eb92c9f133a3f43ffc4d57d0212e4ebd)

Don't test split_ntfs_stream_name

This is a hot code path, and if it has a :, the name will be split later on
anyway.
(This used to be commit 9f7f6b812d89decea1456ccdc37978e645d11a63)

Do not use an unfinished fsp in streams_xattr_open
(This used to be commit 023b313d0d4ed3beb8d77177bc8141cadeb86658)

In streams_xattr_pwrite, base_fsp does not have an fd

These bugs haven't shown up when I did my tests with real xattrs, not with the
xattr_tdb backend. It worked because the stream fsp does has the base file as
fd.
(This used to be commit b7022f8f7bf83c9c5e73e98d1477b7da766e8c5f)

Add some DEBUG
(This used to be commit 8db25aba63b1dffb0dfbc74012c7ebd0ce4d5682)

For pread/pwrite we need to do the setxattr on base_fsp
(This used to be commit d481dddc81699aba79e48bca79bfb10e586b5cbb)

Really remove the _NEXT_ from vfs_xattr_tdb.c

This must have been lost somewhere in my patch-mangling
(This used to be commit c1794375d1fa27d72ec32946bb4bd7b9f96e59fc)

Support XATTR_CREATE and XATTR_REPLACE in vfs_xattr_tdb
(This used to be commit 3509ee597f0977aadd4c70cfe8830a6aa95cd71f)

Complete the ea->xattr rename in vfs_xattr_tdb
(This used to be commit dac468216e7e103b6897f33ec3608412f77265cf)

Fix a segfault

Pointed out by Steven Danneman on irc, thanks!

Jerry, Günther, please check!
(This used to be commit 9e71c89ac648040739ef2161a2e6c4299be1e35b)

Couple of minor fixes for POSIX pathname processing in the
new stream code. (1) In smbd/filename, don't split the name at
':' if we know it's a posix path (this should be parameterized....).
(2). When calling posix_mkdir, we get the flag FILE_FLAG_POSIX_SEMANTICS
passed to open_directory(). I know for a posix client lp_posix_pathnames
should be true (which is checked for in is_ntfs_stream_name() but we
have an explicit flag here, so let's use it.
Jeremy.
(This used to be commit 7bb7a0def6518784befa75e5303289d2b4d36dd4)

Do an explicit init_globals() when restarting for "config backend = registry".

Michael
(This used to be commit 487c43f9284598a3c60b06fc60e5c50844e11b87)

Add a debug message to fetch_reg_values().

Michael
(This used to be commit 239aa59cc1b78f7fb82aa66418cdf92517ebc123)

Add some debugging output to reg_cachehook.c

Michael
(This used to be commit 2a278928805f4497e8afa28bdca433cbedc4a8d7)

Remove the dynamic registry overlay.

It is unnecessary now the dynamic functions have been made
registry backends of their own.

Michael
(This used to be commit e327953bd6b11aeb6f2ae48b49550a942eae8e88)

Add a registry backend perflib that replaces the former dynamic overlay.

Michael
(This used to be commit c3fba415951329ee90c7250b4e8d539f91b227f6)

Some reformatting of current_version_fetch_values().

Michael
(This used to be commit d2e3814db8a4a5f0fc097e9f56753888470ef213)

Add a registry backend current_version that replaces the former dynamic overlay.

Make sure to only respond to the exact current version key since subkeys
are registered by other backends (printing and - soon - perflib).

Michael
(This used to be commit 2c650bf63ccd9dc5dddbf4700831489544ded055)

Reorder the registry key name constants in a more systematic way.

Michael
(This used to be commit faf406a4a1baf2946ea754a0760204748be99541)

Use some consts instead of literal strings for registry keys.

Michael
(This used to be commit 80024f4e1c6594c3038e86a765f763d24fd96b59)

Some reformatting in hkpt_params_fetch_values().

Michael
(This used to be commit f245c4e094ad56080847e286c76976f29c95a221)

Add a registry backend hkpt_params that replaces the former dynamic overlay.

Michael
(This used to be commit 8e8bb6ba120adf9942f612b7fd89bdbced6c1285)

Some reformatting in tcpip_params_fetch_values().

Michael
(This used to be commit 5b3813b233dc2a60e6d5a9942b4044219d800415)

Add a registry backend tcpip_params that replaces the former dynamic overlay.

Michael
(This used to be commit a8a743b693a162954948ca2438ce4b842c5cba30)

Some reformatting in prod_options_fetch_values().

Michael
(This used to be commit 347b9886547516bc2a43190ae7faaf349d2c9d04)

Add a registry backend prod_options that replaces the former dynamic overlay.

Michael
(This used to be commit d9b89e9d30702f64805b3a3a3612066b19c051d1)

Some reformatting in netlogon_params_fetch_values().

Michael
(This used to be commit 24f13fb72e4cbaba48235fa840b2a93bf1c5d9c3)

Rename netlogon_params_fetch_reg_values() to netlogon_params_fetch_values().

Michael
(This used to be commit 54e7c8098565495a833500d4b2a8d5240ed55c82)

In reg_backend_netlogon_params, delegate fetch_subkeys() to regdb.

In order to be able to open the netlogon_params key (the new backend replacing
the former dynamic overlay), certain methods need to be provided. Delegate these
to the regdb backend (like e.g. the smbconf backend does).

Michael
(This used to be commit 9261b2c4bf48e133eecda9ec0095bd8edf20326c)

Fix registering Registry backends in reghook_cache_add().

This was broken in 331c0d6216e1a1607a49ed7eb4078e10138ec16a
(pstring removal).

Michael
(This used to be commit 7d1e986f3a5ab316a8501ddaca1ba1f0867b4531)

Use constant KEY_NETLOGON_PARAMS instead of literal key.

Michael
(This used to be commit daf37c954572bbf652506daae9ff0c9c365b2a9e)

Volker is right: why keep commented out migrated dynamic reg overlays around?...

Michael
(This used to be commit a73b8d16aa0f7a3bb7417e9839e04380e6a68629)

Use c99 struct initializers for REGISTRY_OPS in reg_shares.c

Michael
(This used to be commit 2c4dfd7aaa3c3b384b547451f914a86f59157928)

Use c99 struct initializers for REGISTRY_OPS in reg_printing.c

Michael
(This used to be commit a89bee4139666ba163385c9e7d15fbc5d623ed6f)

Use c99 struct initializers for REGISTRY_OPS in reg_smbconf.c.

Michael
(This used to be commit 96d116b003c1187869cbdbc21274a0b5cb1bf7d6)

Some systems do not have XATTR_ defined
(This used to be commit 2cac1d3919a96c480f34c93d8b9b07782d46ed23)

vfs_streams_depot

Store streams in a file each. Not 100% finished, and not built by default.
(This used to be commit 5f5fc72b01c8e8fc096375c7cb4a97186c387259)

vfs_streams_xattr module

Store streams in posix xattrs. A kludge, as xattrs are limited in many ways,
but it might be a help for some situations.
(This used to be commit 83a805220e52742119546c76a054d50582e33a24)

Add streams support

This is the core of the streams support. The main change is that in
files_struct there is now a base_fsp pointer that holds the main file open
while a stream is open. This is necessary to get the rather strange delete
semantics right: You can't delete the main file while a stream is open without
FILE_SHARE_DELETE, and while a stream is open a successful unlink of the main
file leads to DELETE_PENDING for all further access on the main file or any
stream.
(This used to be commit 6022873cc155bdbbd3fb620689715f07a24d6ed1)

Add "split_ntfs_stream_name()" together with a torture test
(This used to be commit d813bd9e02d9baf916eb96c478be89f0c435e07c)

Add an error mapping for ENOATTR
(This used to be commit 9f0d778490415b05224f36287df999672ee16928)

Add the STREAMINFO vfs call

Based on jpeach's work, modified the streaminfo prototype

Make use of it in trans2.c together with marshall_stream_info()
(This used to be commit c34d729c7c0600a8f11bf7e489a634a4e37fe88e)

Use SAFE_FREE instead of free
(This used to be commit 999647329028147d7c29a3348202641b3e03430e)

Tiny memory leak in lib/version.c

Hi,

while implementing the extra_info version stuff, it occured to me that
samba_version_string() potentially allocates memory which is unused but
never free'd.

If SAMBA_VERSION_VENDOR_PATCH is defined, a second call to asprintf
takes place.  The result is stored in tmp_version.  Afterwards,
samba_version is set to tmp_version without free'ing samba_version
first.  Looks like a simple free(samba_version) is missing.  Patch
against 3.2-test below.

Ok, this only happens once over the lifetime of the application, so it's
no big deal, but I though it doesn't hurt to mention it.

Corinna

* lib/version.c (samba_version_string): Free samba_version
before setting to tmp_version.
(This used to be commit 373a23d48f2dd24e65dbf814ea58b4add2322128)

Fix error return in xattr_tdb_load_attrs
(This used to be commit 64e54ea8f76fe57193955aabc1459fe635233aca)

The remote storage op is gone

Alexander, I think this ok...
(This used to be commit 197b08ad789c4968155f1c711ef43a5383a89289)