Make char* parameters const

- Use const in function signatures whenever appropriate, to help prevent
  errant scribbling on users' buffers. smbc_set_credentials() always acted as
  if its formal parameters were const char *, and changing the formal
  declaration to specify that should not cause any change to the ABI. It is
  still allowable to pass a writable buffer to a function which specifies that
  it will not write to the buffer.

Derrell

prefer gssapi header files from subdirectory

this fixes some compile time noise on FreeBSD 7

WHATSNEW: Update changes since 3.3.0.

Karolin

More warning fixes for Solaris.
Jeremy.

More warning fixes for Solaris.
Jeremy.

Quieten down a boatload of shadowed variable warnings on Solaris.
Makes real problems easier to spot.
Jeremy.

VERSION: Raise version number.

Karolin

WHATSNEW: Updates changes since 3.3.0.

Karolin

all BSDs use this evironment variable

Signed-off-by: Stefan Metzmacher <metze@samba.org>
(cherry picked from commit cd6ae0de52164dd6f50b3614f2511acb3eb15993)

s3/docs: Fix typo in man mount.cifs.

Thanks to Tobias Stoeckmann for reporting!

Karolin
(cherry picked from commit 09a7f93f6be66a8f2a124e49b4effe2b5863f01d)

For some unknown reason ldbrename was not being built in samba3.

Change smbc_set_credentials_with_fallback() (unreleased) to use
const approptiately.
Jeremy.

Fix bug #6133 - Cannot delete non-ACL files on Solaris/ZFS/NFSv4 ACL filesystem.
As the NFSv4 ACL mapping code doesn't map write directory into the DELETE_CHILD
permission bit (which we require before allowing a delete) no one can delete
files without an explicit DELETE_CHILD bit set on the directory. Add this mapping.
Jeremy.

variable grouping: just my OCD desire to keep similar things together

Make libsmbclient work with DFS

Signed-off-by: Derrell Lipman <derrell.lipman@unwireduniverse.com>

s3/docs: Add hint about getfattr in man vfs_acl_xattr.

Karolin
(cherry picked from commit 27307be0f726c899c0e8a0edfdf200650037bb61)

s3/docs: Add man vfs_acl_tdb.

Karolin
(cherry picked from commit 0fd1d4c0906158bab55871d53396bdc7ae533220)

s3/docs: Add man vfs_acl_xattr.

Karolin
(cherry picked from commit 3b0114c0ba676dec2ece96983aff29257399f378)

s3/docs: Fix several typos.

This fixes bug #6127.
Thanks to Justin T Pryzby <justinpryzby [at] users.sourceforge.net>
for the review!

Karolin
(cherry picked from commit 54dbb030e2c6fed0abd0a3a89d08dad9b8a201a1)

WHATSNEW: Update changes since 3.3.0.

Karolin

Fix printf warnings found on systems where time_t <> long int.
Jeremy.

Fix printf type warning.
Jeremy.

Fix a O(n^2) algorithm in regdb_fetch_keys()

Backport the semantics of when to delete alternate data streams on a file truncate.
Needed to fully support stream semantics.
Jeremy.

Fix a buffer handling bug when adding lots of registry keys

This is *ancient*... From 2002, and nobody noticed until someone added lots of
shares using net conf... :-)
(cherry picked from commit 36ae846d15027df5e3a02ffabb08183dad9f6517)

s3-libnet: fix coverity #848 (UNINIT).

Guenther
(cherry picked from commit e9d5405948e35a8eec0b49cc6c066278cb523397)

WHATSNEW: Jeremy fixed CID 602, not Volker.

Thanks Volker for reading the release notes! ;-)

Karolin

WHATSNEW: Update changes since 3.3.0.

Karolin

Gah, typo :-(. Sorry.

Fix coverity CID-602. Possible use of uninitialized var.
Jeremy.

WHATSNEW: Update changes since 3.3.0.

Karolin

Don't miss an absolute pathname as a kerberos keytab path. From Glenn Machin <gmachin@sandia.gov>.
Jeremy.

Fix bug #6117 - Samba 3.3.0: pdbedit -a core dumps.
Jeremy.

s3: remove duplicate prototypes for generated rpc server functions.

Guenther

s3:make: use pidl/pidl for 'make idl'

metze

fix proto.h after idl changes

metze

librpc: rerun 'PIDL=pidl/pidl make idl'

metze

s3:netlogon: implement _netr_LogonGetCapabilities() with NT_STATUS_NOT_IMPLEMENTED

This hopefully fixes bug #6100.

metze
(cherry picked from commit fe417b29bd23b7b935669993e0f01de4c7de2378)

netlogon.idl: add idl for netr_LogonGetCapabilities()

metze
(cherry picked from commit 8339969ed39e4cf3577971869ef2fd127f1c3ae3)

s3-accessmask: re-run make samba3-idl and remove duplicate copies.

Guenther

s3: re-run make idl.

No panic, this just removes the NDR_BUFFER flag from ndr functions when there is
no buffer to process.

Guenther

pidl: add copy of pidl to 3-3-test in order to regenerate librpc code.

pidl's reference is 79190992b3820cd028c961c48bdea9b35baf13c9 here.

Guenther

s3-WHATSNEW: Update WHATSNEW.

Karolin

Make sharesec utility part of the default binary install.

This used to be commit 8f84d42f7.

Fix bug 5920

The length of the memcpy was calculated wrong, r->out.return_authenticator is
a pointer
(cherry picked from commit cb9dd0f58d06a78b7724c3e0ec8f81512edc2d29)

s3-WHATSNEW: Start WHATSNEW for 3.3.1.

Karolin

Attempt to fix bug #6099. According to Microsoft
Windows 7 looks at the negotiate_flags
returned in this structure *even if the
call fails with access denied ! So in order
to allow Win7 to connect to a Samba NT style
PDC we set the flags before we know if it's
an error or not.
Jeremy.

Remove compiler warnings about unknown structure pointer

s3 vfs: Fix SMB_VFS_RECVFILE/SENDFILE macros

remove accidental white space

enums are traditionally signed. Avoid use of high bit in bit flags.

Get rid of the warnings I had for testing

It seems some systems use f_flags instead of f_flag. Use the appropriate one.

Ensure consistency of values on stack (caught by AIX compiler)

ignore new test program executables

Check for the right error return value

Enable total anonymization in vfs_smb_traffic_analyzer, by mapping any user names to the one given by anonymize_prefix, without generating a hash number. This setting is optional and is compatible with the module configuration format of Samba 3.3.

Parameterize in local.h the MAX_RPC_DATA_SIZE, and ensure
that "offered" read from the rpc packet in spoolss is under
that size. Tidyup from analysis from Veracode.
Jeremy.

Noted by Vericode analysis. Correctly use chroot().
Jeremy.

Fix Coverity ID 740 (RESOURCE_LEAK)

Fix Coverity IDs 879 and 880 (RESOURCE_LEAK, REVERSE_INULL)

Fix Coverity ID 742 (RESOURCE_LEAK)

Fix Coverity ID 744

This was marked as a resource leak. This change makes the code a bit clearer
that we always free error_string.

s3:auth: only create_local_token() should add S-1-22-X-Y sids

metze
(cherry picked from commit e7f7ed8bf6281ef01aca53ea44acdd4af4c51aa7)

s3:auth: add S-1-22-X-Y sids to the local token

metze
(cherry picked from commit f14e4d4e54f424c05147cb0e635c9b8930270262)

Fix Coverity ID 745 (RESOURCE_LEAK)

Fix double free caused by incorrect talloc_steal usage.

[Bug 6069] Add a fstatvfs function for libsmbclient

- DFS is not a POSIX feature, so the state of that bit should be zero if DFS
  is *not* supported. Bit feature reversed.

- Added smbc_statvfs function

Derrell

Fix bug #6098 - When the DNS server is invalid, the ads_find_dc() does not work correctly with "security = domain"

1. If DNS server is invalid, the get_sorted_dc_list() is called with
realm(FQDN) and it fails.
2. On the next step, the get_sorted_dc_list() is called with realm(FQDN) again.

I think "again" is wrong place.
On the 2nd step, get_sorted_dc_list() should be called with realm(WORKGROUP).

s3:packaging: fix parsing of include.h in fill-templates

The content has changed slightly in master for the merged build,
and we must be more exact when parsing the lines.

Michael

s3:packaging: fix path to calling update-pkginfo after pushd in fill-templates

Michael

s3:packaging: paths cosmetics in fill-templates

Michael

enable IPv6 support for NetBSD, FreeBSD

most systems include netinet/in.h via netdb.h but *BSD don't.

s3:build: some implementations of "ln" are picky about the order of arguments

Michael

Fix resume handle for _samr_EnumDomainGroups

packaging: make update-pkginfo callable from any directory

Michael
(cherry picked from commit 7ea90835ebf6e2f2c6f325760800d8361d087ede)

packaging: use "find" instead of "du" to find files... :-)

Michael
(This used to be commit 768aa82361a2c4b0aab71433b7593723a2868f0a)
(cherry picked from commit fd7635a1161e25d88a8252a5cf9de70ca2fe4232)

packaging: remove commented out stuff from update-pkginfo

Michael
(cherry picked from commit d714e58ec1f74897cd13e7cd9dd9003600052b45)

packaging: don't clutter the output of fillup-templates  with popd output

Michael
(cherry picked from commit 0c711935d30b500a15ef06fd658d382b2c1e2c84)

packaging: fix fill-templates call to update-pkginfo to pushd appropriately first

Michael
(cherry picked from commit c28d4ae537d9fc0b76eae758d32356eaf1113c89)

packaging: add script fill-templates

This is a standalone-wrapper for update-pkginfo, which is
usually called from create-tarball. It basically repeats
some functionality of create-tarball.

Michael
(cherry picked from commit ca23469a906bab690162184e8d3949897f7b5a67)

[Bug 6069] Add a fstatvfs function for libsmbclient

- Fill in the remainder of the data (or at least as much as we can get) for
  the fstatvfs return value.

Derrell

[Bug 6069] Add a fstatvfs function for libsmbclient

- Reverse the sense of the flags. Since the fstatvfs() function on POSIX-like
  systems would almost certainly indicate case sensitivity (for example),
  leave the bit turned off if the result is the POSIX-like result just as if
  issued on a typical local file system on a POSIX system.

Derrell

s3-spoolss: fix memleak in get_remote_printer_publishing_data().

Guenther

[Bug 6069] Add a fstatvfs function for libsmbclient

- Complete the implementation of the f_flag field. We now return a flag
  indicatin UNIX CIFS, CASE SENSITIVE, and/or DFS support.

Derrell

Revert "s3:winbindd_user: create domain connection in winbindd_fill_pwent if necessary."

This reverts commit 3981603761a1cc6fb74ff135fadf4ab3340be1c5.

I was confused about the real meaning of find_domain_from_name_noinit()
vs. find_domain_from_name(). We don't need the connection established
here, just the domain struct which gets initialized by rescan_trusted_domains().

Sorry for the noise.

Michael

docs: Add man sharesec.

Karolin
(cherry picked from commit cc51cbc862055975bcef016876662576b00e72ef)

[Bug 6069] Add a fstatvfs function for libsmbclient

- Add initial test implementation. This works on a Linux client to determine
  whether a connection's server supports UNIX CIFS. I'm eager to see what the
  build farm has to say about this, specifically how many, if any, hosts don't
  provide a statvfs.h file with the requisite struct statvfs.

Derrell

s3:winbindd_user: create domain connection in winbindd_fill_pwent if necessary.

Calling find_domain_from_name_noinit() might not be enough here.
This makes winbindd_getpwent() behave the same as winbindd_getgrent().

Michael

Signed-off-by: Michael Adam <obnox@samba.org>

s3:winbindd_user: fix a debug message.

find_domain_from_name_noinit() is no longer called only for
name alias support.

Michael

Signed-off-by: Michael Adam <obnox@samba.org>

Fix coverity ID 876 (FORWARD_NULL)

Michael, please check!

Signed-off-by: Michael Adam <obnox@samba.org>

Fix count in _srvsvc_NetDiskEnum()

Without this, remotely adding a share via MMC fails
(cherry picked from commit 197b4c42bbf53347f4953e39099192693d709a7d)

s3-rpcclient: fix getform command asprintf return code.

Guenther

eliminate compiler warnings

mount.cifs: initialize rc to 0 in main

The value of rc in main() isn't initialized in the declaration. This
wasn't a problem before, but Shirish's fakemount patch can make it so
that we return the uninitialized variable if the -n flag is used.

Fix this by initializing rc to 0.

Signed-off-by: Jeff Layton <jlayton@redhat.com>

mount.cifs: add fakemount (-f) and nomtab (-n) flags to mount.cifs

...so that these options work correctly when passed in by mount(8).

s3:net conf: remove check for sharename being a usernam in "net conf addshare"

This is useless and can be overriden by "net conf setparm" anyways.

Michael

docs: fix two typos in the mount.cifs manpage

Michael
(cherry picked from commit 145fe37766cf1ecffb16a03b58b44d08f7ed7558)

Signed-off-by: Michael Adam <obnox@samba.org>

docs: Clarify "ldap ssl" description in man smb.conf.

Mention "ldap ssl ads".

Karolin
(cherry picked from commit 61f1747441ff43c1d9c99bbf69e218c1984834c7)

docs: Add description of "ldap ssl ads" to man smb.conf.

Karolin
(cherry picked from commit 02f13538f00cc8368e3e5e8e510bac468c21089b)