Initialize _domain_list to NULL.

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

Michael

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

Guenther

Remove two unused variables

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

Merge branch 'v3-2-test' of ssh://git.samba.org/data/git/samba into v3-2-test

Added PFIF notice to Samba 3.2 tree

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.

replace svn references with git ones in Readme

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

Avoid use of uninitialized memory

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

Hide streams from the EA API

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

metze

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

Fix get_ea_names_from_file for many EAs

Found by the IBM checker

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!

Add another check for talloc failure to libnet_conf

Michael

Fix formatting of multi_sz registry values.

Don't print only the last component.

Michael

Use talloc_stackframe() for temporary contexts throughout libnet_conf.c

Michael

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

Michael

Fix two debug messages.

Michael

Attempt to fix the darwin build

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

Fix a bug in my bugfix. Thanks to vl for spotting that one.

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!

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!

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.

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

metze

winbindd: remove useless strcpy

metze

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

Move reg_frontend_hilvl.c to reg_dispatcher.c

This is actually the dispatcher to the registered registry backends.

Michael

Extend/fix comments.

Michael

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

Rename reg_db.c to reg_backend_db.c

Michael

Rename reg_shares.c to reg_backend_shares.c

Michael

Rename reg_smbconf.c to reg_backend_smbconf.c

Michael

Rename reg_printing.c to reg_backend_printing.c

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

Michael

Makefile.in: Group together registry related object variables.

Michael

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

Michael

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

Fix some "set but never used" warnings

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.

Activate SMB_QUERY_FILE_STREAM_INFO

gcolley was right, my nt4 does not blue screen with it

Active RAW-STREAMS in make test

Fix valgrind errors

NT_STATUS_OBJECT_NAME_NOT_FOUND also means "no streams around :-)"

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.

Do not use an unfinished fsp in streams_xattr_open

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.

Add some DEBUG

For pread/pwrite we need to do the setxattr on base_fsp

Really remove the _NEXT_ from vfs_xattr_tdb.c

This must have been lost somewhere in my patch-mangling

Support XATTR_CREATE and XATTR_REPLACE in vfs_xattr_tdb

Complete the ea->xattr rename in vfs_xattr_tdb

Fix a segfault

Pointed out by Steven Danneman on irc, thanks!

Jerry, Günther, please check!

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.

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

Michael

Add a debug message to fetch_reg_values().

Michael

Add some debugging output to reg_cachehook.c

Michael

Remove the dynamic registry overlay.

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

Michael

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

Michael

Some reformatting of current_version_fetch_values().

Michael

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

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

Michael

Use some consts instead of literal strings for registry keys.

Michael

Some reformatting in hkpt_params_fetch_values().

Michael

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

Michael

Some reformatting in tcpip_params_fetch_values().

Michael

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

Michael

Some reformatting in prod_options_fetch_values().

Michael

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

Michael

Some reformatting in netlogon_params_fetch_values().

Michael

Rename netlogon_params_fetch_reg_values() to netlogon_params_fetch_values().

Michael

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

Fix registering Registry backends in reghook_cache_add().

This was broken in 331c0d6216e1a1607a49ed7eb4078e10138ec16a
(pstring removal).

Michael

Use constant KEY_NETLOGON_PARAMS instead of literal key.

Michael

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

Michael

Use c99 struct initializers for REGISTRY_OPS in reg_shares.c

Michael

Use c99 struct initializers for REGISTRY_OPS in reg_printing.c

Michael

Use c99 struct initializers for REGISTRY_OPS in reg_smbconf.c.

Michael

Some systems do not have XATTR_ defined

vfs_streams_depot

Store streams in a file each. Not 100% finished, and not built by default.

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.

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.

Add "split_ntfs_stream_name()" together with a torture test

Add an error mapping for ENOATTR

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()

Use SAFE_FREE instead of free

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.

Fix error return in xattr_tdb_load_attrs

The remote storage op is gone

Alexander, I think this ok...

Add get_ea_names_from_file to sanely list posix xattrs

Refactor get_ea_list_from_file to use that.

Make get_ea_value public

Fix the build

... forgot to "git add" it

afs: Use talloc_stackframe() instead of talloc_init()

Thanks to vl for pointing this out.

ntlm_auth: Get rid of statics in manage_squid_ntlmssp_request

ntlm_auth: Rewrite manage_client_ntlmssp_request without statics.

nltm_auth: Use struct ntlm_auth_state in helper functions.

Now rewriting the helpers one after the other can start.

ntlm_auth: Prepare for a deeper rewrite of the helper functions

ntlm_auth: Dynamically allocate the read buffer.

This ports over my changes from Samba4

Link ndr_xattr.o with LIBNDR_GEN_OBJ

It will be used elsewhere (net xattr for example)