WHATSNEW: Start release notes for 3.4.18.

Karolin

VERSION: Bump version number up to 3.4.18.

Karolin

Fix self granting privileges in security=ads.

CVE-2012-2111

WHATSNEW: Release notes 3.4.17.

Karolin

WHATSNEW: Fix typo.

Karolin

WHATSNEW: Start release notes for Samba 3.4.17.

Karolin

VERSION: Bump version up to 3.4.17.

Karolin

rerun 'make samba3-idl'

metze

The last 12 patches address bug #8815 (PIDL based autogenerated code allows
overwriting beyond of allocated array; CVE-2012-1182).

pidl/NDR/Parser: also do range checks on the array size

metze

pidl/NDR/Parser: do array range validation in ParseArrayPullGetLength()

metze

pidl/NDR/Parser: use helper variables for array size and length

metze

pidl/NDR/Parser: remember if we already know the array length

metze

pidl/NDR/Parser: use ParseArrayPullGetLength() to get the number of array elements (bug #8815 / CVE-2012-1182)

An anonymous researcher and Brian Gorenc (HP DVLabs) working
with HP's Zero Day Initiative program have found this and notified us.

metze

pidl/NDR/Parser: split off ParseArrayPullGetSize() and ParseArrayPullGetLength()

metze

pidl/NDR/Parser: simplify logic in DeclareArrayVariables*()

metze

pidl/NDR/Parser: declare all union helper variables in ParseUnionPull()

metze

pidl:NDR/Parser: fix range() for arrays

metze
(cherry picked from commit bea4948acb4bbee2fbf886adeb53edbc84de96da)

pidl: allow foo being on the wire after [length_is(foo)] uint8 *buffer

metze
(cherry picked from commit 92791ce9a8439ac06a22afdbeb0d0fc66c32cb31)

pidl: add support for [string] on fixed size arrays.

midl also supports this:

struct {
long l1;
[string] wchar_t str[16];
long l2;
};

Where the wire size of str is encoded like a length_is() header:
4-byte offset == 0;
4-byte array length;

The strings are zero terminated.

metze
(cherry picked from commit 7ccc9a6ef563cc855752b4e74152420b9be5af43)

WHATSNEW: Prepare release notes for 3.4.16.

Karolin

WHATSNEW: Start release notes for 3.4.16.

Karolin
(cherry picked from commit 6f4316c93105165505bd8975530e01c8e50fb9c0)

VERSION: Bump version up to 3.4.16.

Karolin
(cherry picked from commit 4573fbed636e114d21f6efbb3204ae09dfc9c0c2)

WHATSNEW: Prepare release notes for 3.4.15.

Karolin
(cherry picked from commit 3777f31f377e97e09d6cbe9c65c4c4b79b637da2)

s3/swat: use strlcat instead of strncat to fix build on old Linux distros

SLES 9's glibc for example had weird macros where the use of strncat resulted
in the use of strcat which we don't allow.

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Autobuild-User: Björn Jacke <bj@sernet.de>
Autobuild-Date: Thu Aug  4 17:50:24 CEST 2011 on sn-devel-104
(cherry picked from commit d3b4d75364210e2d2a4a1cd806f28b0021f22909)

Fix bug #8362 (build issue on old glibc systems).
(cherry picked from commit 87fa72a5202fe3780d4a61289bf755027cd078f4)
(cherry picked from commit 552ccc6588b0744ae9b3731b1406749baea03d5a)
(cherry picked from commit b941edf9c20bd6a92aec2a62be5b830db1194ce8)

s3:web/swat: use strtoll() instead of atoi/atol/atoll

This is more portable, as we have a strtoll replacement
in lib/replace.

metze

Autobuild-User: Stefan Metzmacher <metze@samba.org>
Autobuild-Date: Sat Aug  6 11:55:45 CEST 2011 on sn-devel-104
(cherry picked from commit a6be0820d09b3f3eabfbb5f4356add303aa8a494)

Fix bug #8347 (CVE-2011-2522 regression for HP-UX, AIX and OSF).
(cherry picked from commit ac5d8c0148e10a3a0af9e1dc0849bb6920c26ad7)

s3:nmbd_packets: return the used number of sockets in create_listen_fdset() (bug #8276)

Fix bug #7949 (DoS in Winbind and smbd with many file descriptors open)
(commit feb3fcd0fa4bda0967b881315595d7702f4d1752) changed the bahavior,
so that we skipped some sockets.

This should work for v3-4-test.

metze
(cherry picked from commit 11b4dec29c9306531e73d5f4c12f89934dd538b4)

s3: increase the log level for missing PIDs on SIGCHLD

Since the fix for bso#7836, the parent smbd is responsible for
maintaining an up-to-date printcap cache. It does this by forking a
child process to asynchronously fetch printcap data from CUPS.

When the child process exits after fetching all printcap data, the
parent smbd is sent SIGCHLD. This triggers smbd_sig_chld_handler() which
looks for the exited process PID on a "children" list.
Child smbd process PIDs are added to the "children" list to ensure
cleanup on unclean shutdown and log level change notification messages.

Printcap update process PIDs are not added to the list as they do not
maintain any state that requires cleanup, nor do they wait on tevent for
messages.

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Thu Feb 17 11:11:45 CET 2011 on sn-devel-104
(cherry picked from commit 9c12232f1ae36e00d04114ad73edd8ba3c2c6a5c)

Fix bug #8269 (smbd spams log with "Could not find child X -- ignoring"
messages).
(cherry picked from commit ba118ac287d49267dd2f346d4ddd2e590ebbe653)
(cherry picked from commit c943af7ff13c64343e5246b27a7416627bffd365)

s3-libnet: fix bug #6364: Pull realm from supplied username on libnet join
(cherry picked from commit 9a172dcd2e7bf91d78885325bce7a19e937b48af)

s3-printing: remove duplicate cups response processing code

There is currently a lot of duplicate code included for processing
responses to CUPS_GET_PRINTERS and CUPS_GET_CLASSES requests. This
change splits this code into a separate function.

Signed-off-by: Günther Deschner <gd@samba.org>
(cherry picked from commit a3635edc96e481f1125db1c26e33cae5b22a01a4)

s3-printing: use printcap IDL for IPC

Use printcap IDL for marshalling and unmarshalling messages between cups
child and parent smbd processes. This simplifies the IPC and ensures
the parent is notified of cups errors encountered by the child.

https://bugzilla.samba.org/show_bug.cgi?id=7994

Signed-off-by: Günther Deschner <gd@samba.org>
(cherry picked from commit e5a0ac40d7467bb6ce07d942fdadb43f1ec81bfc)

idl: define printcap IPC message format

Signed-off-by: Günther Deschner <gd@samba.org>
(cherry picked from commit 9a107d2d65d3080aebe441b8b93cf17abeb092f7)

s3-printing: an empty cups printer list is treated as an error

cups_async_callback() is called to receive new printcap data from a
child process which requests the information from cupsd.
Newly received printcap information is stored in a temporary printcap
cache (tmp_pcap_cache). Once the child process closes the printcap IPC
file descriptor, the system printcap cache is replaced with the newly
populated tmp_pcap_cache, however this only occurs if tmp_pcap_cache is
non null (has at least one printer).

If the printcap cache is empty, which is the case when cups is not
exporting any printers, the printcap cache is not replaced resulting in
stale data.

Signed-off-by: Günther Deschner <gd@samba.org>
(cherry picked from commit 52bac5ffeea8ecbd2a5ecca023b3e2014c1350da)

s3-printing: remove pcap_cache_loaded asserts

pcap_cache_loaded() assertions were added to the (re)load_printers()
functions, to ensure the caller had called pcap_cache_reload() prior to
reloading printer shares.

The problem is, pcap_cache_loaded() returns false if the the pcap_cache
contains no printer entries. i.e. pcap_cache_reload() has run but not
detected any printers.

Remove these assertions, correct call ordering is already enforced.

Signed-off-by: Günther Deschner <gd@samba.org>
The last 3 patches address bug #7836 (A newly added printer isn't visbile to
clients).
(cherry picked from commit da9b2d9d58193ed3da36c2f8ff1e41a1e743ba07)

Revert "Revert "s3-printing: update parent smbd pcap cache""

This reverts commit ad450870eacb114b3f15941a4478ba25701e035a.

Signed-off-by: Günther Deschner <gd@samba.org>
(cherry picked from commit ee094b53d7fd9c46bca57b9815993282c63af00c)

Revert "Revert "s3-printing: reload shares after pcap cache fill""

This reverts commit 36ea03bbe28122ce03de4969e254dd276cfe5a79.

Signed-off-by: Günther Deschner <gd@samba.org>
(cherry picked from commit 3e1969d229dff2921b43189969dfbe522b167cc2)

WHATSNEW: Start release notes for 3.4.15.

Karolin

VERSION: Bump version up to 3.4.15.

Karolin

s3-swat: Fix typo.

Thanks to Simo for reporting!

Karolin

s3 swat: Create random nonce in CGI mode

In CGI mode, we don't get access to the user's password, which would
reduce the hash used so far to parameters an attacker can easily guess.
To work around this, read the nonce from secrets.tdb or generate one if
it's not there.
Also populate the C_user field so we can use that for token creation.

Signed-off-by: Kai Blin <kai@samba.org>
The last 12 patches address bug #8290 (CSRF vulnerability in SWAT).
This addresses CVE-2011-2522 (Cross-Site Request Forgery in SWAT).

s3 swat: Add time component to XSRF token

Signed-off-by: Kai Blin <kai@samba.org>

s3 swat: Add XSRF protection to printer page

Signed-off-by: Kai Blin <kai@samba.org>

s3 swat: Add XSRF protection to password page

Signed-off-by: Kai Blin <kai@samba.org>

s3 swat: Add XSRF protection to shares page

Signed-off-by: Kai Blin <kai@samba.org>

s3 swat: Add XSRF protection to globals page

Signed-off-by: Kai Blin <kai@samba.org>

s3 swat: Add XSRF protection to wizard page

Signed-off-by: Kai Blin <kai@samba.org>

s3 swat: Add XSRF protection to wizard_params page

Signed-off-by: Kai Blin <kai@samba.org>

s3 swat: Add XSRF protection to viewconfig page

Signed-off-by: Kai Blin <kai@samba.org>

s3 swat: Add XSRF protection to status page

Signed-off-by: Kai Blin <kai@samba.org>

s3 swat: Add support for anti-XSRF token

Signed-off-by: Kai Blin <kai@samba.org>

s3 swat: Allow getting the user's HTTP auth password

Signed-off-by: Kai Blin <kai@samba.org>

s3 swat: Fix possible XSS attack (bug #8289)

Nobuhiro Tsuji of NTT DATA SECURITY CORPORATION reported a possible XSS attack
against SWAT, the Samba Web Administration Tool. The attack uses reflection to
insert arbitrary content into the "change password" page.

This patch fixes the reflection issue by not printing user-specified content on
the website anymore.

Signed-off-by: Kai Blin <kai@samba.org>

WHATSNEW: Update release notes.

Karolin

WHATSNEW: Start release notes for Samba 3.4.14.

Karolin
(cherry picked from commit 2c9a3dbdb18f8773754d2cc417c61f5dfd452008)

VERSION: Bump version number up to 3.4.14.

Karolin
(cherry picked from commit 2c72a084ec5eb8d368e34962b39278eb3b2176c6)

WHATSNEW: Update changes since 3.4.12.

Karolin
(cherry picked from commit b5e5510ea62af1df31b5f06c5cf1029217b648e1)

s3: Fix bug 8099 - setpwent() actually does endpwent() on FreeBSD
(cherry picked from commit 2167ac2cd42c9ed5aaae0086dbd27e29d1d77686)
(cherry picked from commit fd387cd0bf186a94b64e8c2085c943992629e5af)

WHATSNEW: Prepare 3.4.13 release notes.

Karolin
(cherry picked from commit 0517118cbfe7455ba288d6c59cb5ed7f22274b35)

s3-cli_pipe: fix timeout in rpc_pipe_open_tcp_port().

Make sure we use a timeout of 60 seconds, not 60 milliseconds...

This prevented us from successfully using the ncacn_ip_tcp client in a lot of
places, I guess.

Guenther

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Wed Apr 13 18:59:19 CEST 2011 on sn-devel-104
(cherry picked from commit 4b3fe5247a6e16b1ad9f05269e9aa00e3120e36a)

Fix bug #8085 - incorrect timeout handling in ncacn_ip_tcp client code.
(cherry picked from commit d7d39c723e1855a3d18813e8a79fcca9770b0142)
(cherry picked from commit f7a175f47ee65c58363615541577db65d8b9fa76)

Allow NULL queue to writev_send

Fix bug #8086 - null pointer reference crashes winbind.
(cherry picked from commit 7a801a842c7c6636c654489cb53bb8271c6927c8)

WHATSNEW: Start 3.4.13 release notes.

Karolin

VERSION: Bump version number up to 3.4.13.

Karolin

WHATSNEW: Fix typo.

Karolin

VERSION: Bump version number up to 3.4.12.

Karolin

WHATSNEW: Prepare 3.4.12 release notes.

Karolin

Fix denial of service - memory corruption.

CVE-2011-0719

Fix bug #7949 (DoS in Winbind and smbd with many file descriptors open).

All current released versions of Samba are vulnerable to
a denial of service caused by memory corruption. Range
checks on file descriptors being used in the FD_SET macro
were not present allowing stack corruption. This can cause
the Samba code to crash or to loop attempting to select
on a bad file descriptor set.

A connection to a file share, or a local account is needed
to exploit this problem, either authenticated or unauthenticated
(guest connection).

Currently we do not believe this flaw is exploitable
beyond a crash or causing the code to loop, but on the
advice of our security reviewers we are releasing fixes
in case an exploit is discovered at a later date.

WHATSNEW: Fix typo.

Karolin
(cherry picked from commit dff57d78ee0c5528c67a5a86bb356e9729107b11)

WHATSNEW: Add changes since 3.4.10.

Karolin
(cherry picked from commit bcdfa183c1de202f64805942041dd182f6c013ea)

s3: Fix connecting to port-139 only servers

When the TCP RST came before the 5 msecs timeout kicked in, we
viewed this as final, as state->req_139 was not set yet.

Fix bug introduced by a fix for bug #7881 (winbind flaky against w2k8).
(cherry picked from commit f2a19b87725f9318e983dff6358a3eee721bff08)
(cherry picked from commit 569be63e727e69e7e52ec39f40e60903c6826614)

WHATSNEW: Start release notes for Samba 3.4.11.

Karolin
(cherry picked from commit 716426b31f24923e94099e99580ae7750eb5e498)

VERSION: Raise version number up to 3.4.11.

Karolin
(cherry picked from commit fa8af888f7c701ec95febdb90c57aee1e418ad2e)

WHATSNEW: Add major enhancements.

Karolin
(cherry picked from commit 0d069e7aa3c3075cee6e698d9f7aa734826916ad)

WHATSNEW: Update changes since 3.4.9.

Karolin
(cherry picked from commit a215aad589fb30c31e52a34bc5fdbbcbfb91de7f)

WHATSNEW: Start to add changes since 3.4.9.

Karolin
(cherry picked from commit 948a7b8cef5291ae11ae3fe875c1d53ad9b3b91a)

s3-rpcclient: Fix bug #7880: cmd_spoolss_deletedriver() returned without checking all architectures.

Continues now with next architecture if no driver is available.

Because of the broken behavior of the rpccli_*() functions,
we need special error code handling.

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

s3-dns Don't use DELEG_FLAG in DNS update, Windows 2008R2 does not like it (cherry picked from commit 280caa6b3bb1199939f9349ea5a436a491c81791)

The last 2 patches address bug #7356 (net ads dns register fails in 2008 R2
domain).
(cherry picked from commit 6857b749229cc72c604ab5646a4bae5f09b72e11)
(cherry picked from commit 7cca44fa97762ccde7166a80bec91a7849f029c1)

s3-dns Don't use SEQUENCE_FLAG in DNS update, Windows 2008R2 does not like it

Andrew Bartlett
(cherry picked from commit 0f1cc889a26477e9a98629f120fe5890b2e106fa)
(cherry picked from commit 2b463484cc7bb80cdfb6727ab9e5a873faff5ec8)
(cherry picked from commit 1d2f3742fcb47e4a13e21b8e3b7e22a9ddeba49c)

WHATSNEW: Start release notes for Samba 3.4.10.

Karolin
(cherry picked from commit 1d1ea2dfaf17c65d0630d982fdf46fa0e805ab09)

VERSION: Bump version number.

Karolin
(cherry picked from commit f4762c74c0be61cd3c733d82767878fbb25835ac)

s3-nmbd: Fix bug #7875

nmbd --port didn't work
(cherry picked from commit 79280c99f67c3a3bfb1873b373ec181fa402f18c)
(cherry picked from commit 2b1a5fad6c681d8ff8e592ee92d8251040c0760d)

s3:lib/netapi: don't set SAMR_FIELD_FULL_NAME if we just want to set the account name (bug #7896)

metze

Autobuild-User: Stefan Metzmacher <metze@samba.org>
Autobuild-Date: Thu Dec 30 18:09:13 CET 2010 on sn-devel-104
(cherry picked from commit f1d15ea54c313e71fc032b2ed191bdecad868858)
(cherry picked from commit c6a0971b3790253a906b370562237479d273bb94)
(cherry picked from commit b46ec13a9a548cf1bc4ada47751f9e6945306bc7)

s3: Fix bug 7066 -- wbcAuthenticateEx gives unix times

We might eventually want to change this, but right now we get unix times
out of the winbind pipe struct
(cherry picked from commit 993923880e213136de89b5b8d59f6f32a51b94b7)
(cherry picked from commit 15075858886ee208f800f9bfdcfaf6a56d8653de)

ѕ3/configue: set Tru64 cc's PIC switch right (none)

-fPIC made shared library builds fail there

Fixes #7821

(cherry picked from commit dbcf73c45782c310cb7ff1f2177d410399e2f06d)
(cherry picked from commit 83eb2e9aef40e5e838d2654298e281ad3ec98af3)
(cherry picked from commit d28ba1d46c226c8c611816393cdf8eba393300f5)

Fix bug #7892 - open_file_fchmod() leaves a stale lock.
(cherry picked from commit 023aa6f4aae29ba22b3d42c815027ba6a66f8ee2)

s3: Use smbsock_any_connect in winbind

The last 8 patches address bug #7881 (winbind flaky against w2k8).
(cherry picked from commit 1a929d8f4c6bae302e5cc53d81117a05eae5fede)

s3: Retry *SMBSERVER in nb_connect
(cherry picked from commit ee2534c18b5afa609ff17d9da7ea10bcf7654fc6)

s3: Add smbsock_any_connect
(cherry picked from commit 198b6d673787ee40c0ac389ece99cee1844dd585)

s3: Add an async smbsock_connect

This connects to 445 and after 5 milliseconds also to 139. It treats a netbios
session setup failure as equivalent as a TCP connect failure. So if 139 is
faster but fails the nb session setup, the 445 still has the chance to succeed.
(cherry picked from commit 35bbc2231760badaf0debc9f8f39ebdf00cfe8ad)

v3-4-test: Pull in tevent_req_poll_ntstatus from master
(cherry picked from commit 9b79de1553cfa57d24d1ac0316b814933ef3d33d)

s3: Add async cli_session_request

This does not do the redirects, but I think that might be obsolete anyway
(cherry picked from commit e2296e23a8546e249d1b26f4da6277792923bef4)

v3-4-test: Pull in read_smb_send from master
(cherry picked from commit 02c4649674d3bd0f54e71910f11d6aff2cdb6c9d)

s3: Add some const to name_mangle()
(cherry picked from commit 56c760ab41b9b4cb9680d873b8f9955be21434f4)

s3: Make winbind recover from a signing error

When winbind sees a signing error on the smb connection to a DC (for whatever
reason, our bug, network glitch, etc) it should recover properly. The "old"
code in clientgen.c just closed the socket in this case. This is the right
thing to do, this connection is spoiled anyway. The new, async code did not do
this so far, which led to the code in winbindd_cm.c not detect that we need to
reconnect.

Fix bug #7800 (winbind does not recover from smb signing errors).
(cherry picked from commit 8c2493ff2e646928035ec7296f4451f09390f6aa)

s3: Stop using the write cache after an oplock break

Fix bug #7715 (Setting Samba Write Cache Size Can Cause File Corruption).
(cherry picked from commit 9f8292e5f765dff586bfbb261b54da4d4b27a837)
(cherry picked from commit e18ef6cdf042a73e7f08b792e4a9901b071b1f67)

Fix bug 7636 - winbind internal error, backtrace.

Jeremy.
(cherry picked from commit 995e7e500327e662b7ef2b37c83c92e75f2360bf)

s3-printing: fix BUG 7280 - auto printers not loading with registry config
(cherry picked from commit 9c3537b6d8b2ea36265d3a69f7e90278cb86dbfb)

Fix bug 7590 - offline login fails because winbind deletes cache on every startup.

Sync lib/tdb_validate.c with the change in current master.
Change tdb_validate_open() to always use O_RDWR instead of O_RDONLY,
as (from the bug report): "db_check() will always return failure for a read-only database.
Silently, without any log output, when _tdb_lockall() fails."

Jeremy.
(cherry picked from commit 39cb903463d8a3fcabd9e148112bf5cf81744130)
(cherry picked from commit 9812b1a601c9ccc2a3e6c06b98c32730a32c787a)

rerun: make samba3-idl

metze
(cherry picked from commit 9b9b46252ecf426467205908cb7ba487b7774b5b)

pidl: Samba3/ClientNDR - Correctly copy arrays, if r.out.size < r.in.size.

metze

Signed-off-by: Andreas Schneider <asn@samba.org>
(similar to commit 33d1879d5b50e2d98c1bb13b835e7cfb178e3336)
(similar to commit d1e92cd2944983ecabd0511ff7c8221c1033a3a8)
Fixes bug #7607.
(cherry picked from commit d04e5c4d0ae4bb20497748771aa1467cc14f2265)

Fix bug #7617 - smbd coredump due to uninitialized variables in the performance counter code.

In the file rpc_server.c, function _winreg_QueryValue()

uint8_t *outbuf

Should be :

uint8_t *outbuf = NULL;

As it is later freed by

      if (free_buf) SAFE_FREE(outbuf);

in some cases, this frees the unintialized outbuf, which causes a coredump.
(cherry picked from commit 84fd910c347ddfad6f01edbe7f6e25546c8382ee)
(cherry picked from commit 80e65236158d6f1690bf9f153c0eb12d81d56b8a)
(cherry picked from commit 78d1a15920de4ef3f230511257ee2f334f89a642)

s3-winbind: Fix Bug #7568: Make sure cm_connect_lsa_tcp does not reset the secure channel.

This is an important fix as the following could and is happening:

* winbind authenticates a user via schannel secured netlogon samlogonex call,
current secure channel cred state is stored in winbind state, winbind
sucessfully decrypts session key from the info3

* winbind sets up a new schannel ncacn_ip_tcp lsa pipe (and thereby resets the
secure channel on the dc)

* subsequent samlogonex calls use the new secure channel creds on the dc to
encrypt info3 session key, while winbind tries to use old schannel creds for
decryption

Guenther
(cherry picked from commit be396411a4e1f3a174f8a44b6c062d834135e70a)
(cherry picked from commit e647f5b5409502ec329e24f09202b036cfb357ae)