samba.git
12 years agoWHATSNEW: Add changes since 3.5.10. samba-3.5.11
Karolin Seeger [Wed, 3 Aug 2011 18:20:58 +0000 (20:20 +0200)]
WHATSNEW: Add changes since 3.5.10.

Karolin
(cherry picked from commit 529bfe1d43fc8f6c7a1ef93d437f40cd89b1ab63)

12 years agoFix bug 7462 - Non-standard SA_RESETHAND is used in ...lib/tevent/tevent_sig
Jeremy Allison [Tue, 2 Aug 2011 18:49:46 +0000 (20:49 +0200)]
Fix bug 7462 - Non-standard SA_RESETHAND is used in ...lib/tevent/tevent_sig

Make SA_RESETHAND conditional on its existance.

Autobuild-User: Jeremy Allison <jra@samba.org>
Autobuild-Date: Mon Aug  1 22:03:45 CEST 2011 on sn-devel-104
(cherry picked from commit 0c67efdd68b9808542c090b9fd9920e4e37d85d0)
(cherry picked from commit 490986add9d5e80b24e90dbfe3e3ef23ce5584a0)

12 years agos3: Test for "__attribute__((destructor))"
Volker Lendecke [Mon, 10 May 2010 09:53:03 +0000 (11:53 +0200)]
s3: Test for "__attribute__((destructor))"
(cherry picked from commit 36e736871e28665ffcbbc4d0c87e1a2b60fcf0e0)

Fix bug #8322 (HAVE_FUNCTION_ATTRIBUTE_DESTRUCTOR is missing from 3.5.x).
(cherry picked from commit c69ab4252d9aea274f44f96b1e3f41ca57185987)

12 years agos3:librpc/gen_ndr: regen after wbint.idl changes
Stefan Metzmacher [Mon, 4 Jul 2011 09:47:24 +0000 (11:47 +0200)]
s3:librpc/gen_ndr: regen after wbint.idl changes

metze

Part of a fix for bug #7841 (WINBINDD_LOOKUPRIDS asks the wrong domain).
(cherry picked from commit 12a4b5633d5c3488f48f3e7a5084402f6eeeb9c5)

12 years agos3:nmbd_subnetdb: close all sockets attached to a subnet in close_subnet() (bug ...
Stefan Metzmacher [Thu, 30 Jun 2011 08:09:56 +0000 (10:09 +0200)]
s3:nmbd_subnetdb: close all sockets attached to a subnet in close_subnet() (bug #8276)

metze
(cherry picked from commit 75e9f2110876137a57632d223248ac51dbfc4569)
(cherry picked from commit 48355dae98885ca0d44dbf4206a0bdf16c64fced)

12 years agos3:nmbd_packets: make sure create_listen_fdset() returns initialized data (bug #8276)
Stefan Metzmacher [Thu, 30 Jun 2011 07:56:06 +0000 (09:56 +0200)]
s3:nmbd_packets: make sure create_listen_fdset() returns initialized data (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-5-test.

metze
(cherry picked from commit 9d5738f779b803fb257537d6308a5f34625cb1b0)

12 years agos3:smbldap: make smbldap_connect_system self contained
Gregor Beck [Tue, 21 Jun 2011 06:16:56 +0000 (08:16 +0200)]
s3:smbldap: make smbldap_connect_system self contained

The last 5 patches address bug #8253 (winbindd panics if verify_idpool() fails).
(cherry picked from commit 1da14b93bc664948699f62cca2fc777c6f7523c9)

12 years agos3:smbldap: add a destructor to smbldap_state, just in case
Gregor Beck [Tue, 21 Jun 2011 06:06:28 +0000 (08:06 +0200)]
s3:smbldap: add a destructor to smbldap_state, just in case
(cherry picked from commit 6a3869da05b0d0e4d47db2502489de359d5e7e45)

12 years agos3:smbldap: let smbldap_free_struct do what it claims to
Gregor Beck [Tue, 21 Jun 2011 06:02:53 +0000 (08:02 +0200)]
s3:smbldap: let smbldap_free_struct do what it claims to
(cherry picked from commit df03f6c2c98f65bf9656d27e1cc9dc72cd587e31)

12 years agos3:smbldap: free the idle event scheduled in smbldap_open in smbldap_close
Gregor Beck [Tue, 21 Jun 2011 06:00:59 +0000 (08:00 +0200)]
s3:smbldap: free the idle event scheduled in smbldap_open in smbldap_close
(cherry picked from commit 70856a728a0be1c97e9e13382cd2d880450e07c4)

12 years agos3:smbldap: use smbldap_state as memory context for idle event
Gregor Beck [Tue, 21 Jun 2011 05:51:41 +0000 (07:51 +0200)]
s3:smbldap: use smbldap_state as memory context for idle event

ensure the event is canceled if the smbldap_state gets freed
this fixes a panic of winbindd if verify_idpool fails
(cherry picked from commit 3d78bea9ac27c3f6c98561e287add632a17ce747)

12 years agos3: explicitly pass domain_sid to wbint_LookupRids() (bug #7841)
Volker Lendecke [Mon, 27 Jun 2011 12:34:39 +0000 (14:34 +0200)]
s3: explicitly pass domain_sid to wbint_LookupRids() (bug #7841)
(cherry picked from commit 1a8155d347e2a8de3432ce0fe99d598c25c2bafb)

12 years agoFix bug #8254 - "acl check permissions = no" does not work in all cases
Jeremy Allison [Thu, 23 Jun 2011 21:42:27 +0000 (14:42 -0700)]
Fix bug #8254 - "acl check permissions = no" does not work in all cases

Move lp_acl_check_permissions() into can_delete_file_in_directory()
where it makes sense. Remove ACL check when requesting DELETE_ACCESS
when lp_acl_check_permissions is false.

Thanks to John Janosik @ IBM for noticing this.
(cherry picked from commit 83357ecf6adafe3d23ada705e79c3af25ad4e734)

12 years agos3: increase the log level for missing PIDs on SIGCHLD
David Disseldorp [Wed, 16 Feb 2011 16:23:25 +0000 (17:23 +0100)]
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)

12 years agoWHATSNEW: Formatting.
Karolin Seeger [Fri, 24 Jun 2011 19:16:17 +0000 (21:16 +0200)]
WHATSNEW: Formatting.

Karolin
(cherry picked from commit efd437efea9096837f3f45b47daff3e147cddea8)

12 years agos3-WHATSNEW 3.5.9 Add information on kerberos change
Andrew Bartlett [Tue, 14 Jun 2011 12:04:11 +0000 (22:04 +1000)]
s3-WHATSNEW 3.5.9 Add information on kerberos change
(cherry picked from commit eb7c10ea16ff7db34d8ab71306c7d5d298df8d40)

12 years agos3: Fix bug 8238 -- KB2536276 prevents access to shares
Volker Lendecke [Thu, 16 Jun 2011 20:20:49 +0000 (22:20 +0200)]
s3: Fix bug 8238 -- KB2536276 prevents access to shares

Without this we were not sending the workgroup name in the negprot reply if
plain text passwords are used.
(cherry picked from commit d42bf679b0807ebc47f43c62d4b883e0b5096abb)

12 years agodocs: fix the missing parameter description section in the smb.conf manpage
Michael Adam [Tue, 5 Apr 2011 21:07:01 +0000 (23:07 +0200)]
docs: fix the missing parameter description section in the smb.conf manpage

The smb.conf (5) manpage recently sometimes failed to contain the
contents of the description of each parameter section. The reason
was a unreliable chain of dependencies in the Makefile.

The error can be reproduced by touching manpages-3/smb.conf.5.xml
and then building the manpages.
Then smb.conf.5.xml is newer than any of the smbdotconf/*/*.xml
files and hence the intermediate inexistent parameters.*.xml
don't get generated.

This patch fixes this problem by introducing a phony "parameters"
target referencing the parameters.*.xml targets, so that they
get build unconditionally.

Fix bug #7997 (smb.conf.5 manpage truncated in 3.5.8).
(cherry picked from commit 08e64ab50b9202ccbc6c0b397f9ac8cc467c2254)

12 years agolibreplace: include sys/file.h only when available
Björn Jacke [Sun, 30 May 2010 19:52:39 +0000 (21:52 +0200)]
libreplace: include sys/file.h only when available

thanks to Joachim Schmitz <schmitz@hp.com>. This fixes #7460.
(cherry picked from commit a33b6032beb45f7ba07432899236fccb133a6dfc)

12 years agos3-docs Add documentation for 'client use spnego principal'
Andrew Bartlett [Tue, 14 Jun 2011 12:16:02 +0000 (22:16 +1000)]
s3-docs Add documentation for 'client use spnego principal'
(cherry picked from commit 4829da5bd4989b7e4b7e858af1770f13d9d2e647)

12 years agoWHATSNEW: Start release notes for 3.5.11.
Karolin Seeger [Tue, 26 Jul 2011 18:39:28 +0000 (20:39 +0200)]
WHATSNEW: Start release notes for 3.5.11.

Karolin

12 years agoVERSION: Bump version up to 3.5.11.
Karolin Seeger [Tue, 26 Jul 2011 18:36:37 +0000 (20:36 +0200)]
VERSION: Bump version up to 3.5.11.

Karolin

12 years agos3-swat: Fix typo. samba-3.5.10
Karolin Seeger [Sun, 24 Jul 2011 19:09:38 +0000 (21:09 +0200)]
s3-swat: Fix typo.

Thanks to Simo for reporting!

Karolin

12 years agos3 swat: Create random nonce in CGI mode
Kai Blin [Tue, 12 Jul 2011 06:08:24 +0000 (08:08 +0200)]
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).

12 years agos3 swat: Add time component to XSRF token
Kai Blin [Sat, 9 Jul 2011 07:52:07 +0000 (09:52 +0200)]
s3 swat: Add time component to XSRF token

Signed-off-by: Kai Blin <kai@samba.org>
12 years agos3 swat: Add XSRF protection to printer page
Kai Blin [Fri, 8 Jul 2011 13:06:13 +0000 (15:06 +0200)]
s3 swat: Add XSRF protection to printer page

Signed-off-by: Kai Blin <kai@samba.org>
12 years agos3 swat: Add XSRF protection to password page
Kai Blin [Fri, 8 Jul 2011 13:05:38 +0000 (15:05 +0200)]
s3 swat: Add XSRF protection to password page

Signed-off-by: Kai Blin <kai@samba.org>
12 years agos3 swat: Add XSRF protection to shares page
Kai Blin [Fri, 8 Jul 2011 13:04:48 +0000 (15:04 +0200)]
s3 swat: Add XSRF protection to shares page

Signed-off-by: Kai Blin <kai@samba.org>
12 years agos3 swat: Add XSRF protection to globals page
Kai Blin [Fri, 8 Jul 2011 13:04:12 +0000 (15:04 +0200)]
s3 swat: Add XSRF protection to globals page

Signed-off-by: Kai Blin <kai@samba.org>
12 years agos3 swat: Add XSRF protection to wizard page
Kai Blin [Fri, 8 Jul 2011 13:03:44 +0000 (15:03 +0200)]
s3 swat: Add XSRF protection to wizard page

Signed-off-by: Kai Blin <kai@samba.org>
12 years agos3 swat: Add XSRF protection to wizard_params page
Kai Blin [Fri, 8 Jul 2011 13:03:15 +0000 (15:03 +0200)]
s3 swat: Add XSRF protection to wizard_params page

Signed-off-by: Kai Blin <kai@samba.org>
12 years agos3 swat: Add XSRF protection to viewconfig page
Kai Blin [Fri, 8 Jul 2011 13:02:53 +0000 (15:02 +0200)]
s3 swat: Add XSRF protection to viewconfig page

Signed-off-by: Kai Blin <kai@samba.org>
12 years agos3 swat: Add XSRF protection to status page
Kai Blin [Fri, 8 Jul 2011 10:58:53 +0000 (12:58 +0200)]
s3 swat: Add XSRF protection to status page

Signed-off-by: Kai Blin <kai@samba.org>
12 years agos3 swat: Add support for anti-XSRF token
Kai Blin [Fri, 8 Jul 2011 10:57:43 +0000 (12:57 +0200)]
s3 swat: Add support for anti-XSRF token

Signed-off-by: Kai Blin <kai@samba.org>
12 years agos3 swat: Allow getting the user's HTTP auth password
Kai Blin [Fri, 8 Jul 2011 10:56:21 +0000 (12:56 +0200)]
s3 swat: Allow getting the user's HTTP auth password

Signed-off-by: Kai Blin <kai@samba.org>
12 years agos3 swat: Fix possible XSS attack (bug #8289)
Kai Blin [Thu, 7 Jul 2011 08:03:33 +0000 (10:03 +0200)]
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>
CVE-2011-2694.

12 years agoWHATSNEW: Update release notes.
Karolin Seeger [Sun, 24 Jul 2011 20:00:16 +0000 (22:00 +0200)]
WHATSNEW: Update release notes.

Karolin

12 years agoWHATSNEW: Start release notes for 3.5.10.
Karolin Seeger [Tue, 14 Jun 2011 17:11:21 +0000 (19:11 +0200)]
WHATSNEW: Start release notes for 3.5.10.

Karolin
(cherry picked from commit a32956ad21b70d10a5b2a8516e39cbd35aa09045)

12 years agoVERSION: Bump version up to 3.5.10.
Karolin Seeger [Tue, 14 Jun 2011 17:08:58 +0000 (19:08 +0200)]
VERSION: Bump version up to 3.5.10.

Karolin
(cherry picked from commit 767cf9e13421c00a658dac93ef77353587539344)

12 years agoWHATSNEW: Update changes since 3.5.8. samba-3.5.9
Karolin Seeger [Tue, 14 Jun 2011 11:15:37 +0000 (13:15 +0200)]
WHATSNEW: Update changes since 3.5.8.

Karolin
(cherry picked from commit d1880d237bdf79b036623ebf5ae477838c9482c9)

12 years agoFix re-opened bug 8083 - "inherit owner = yes" doesn't interact correctly with vfs_ac...
Jeremy Allison [Tue, 7 Jun 2011 19:36:24 +0000 (12:36 -0700)]
Fix re-opened bug 8083 - "inherit owner = yes" doesn't interact correctly with vfs_acl_xattr or vfs_acl_tdb module.

Fix incorrect interaction when all of

"inherit permissions = yes"
"inherit acls = yes"
"inherit owner = yes"

are set. Found by Björn Jacke. Thanks Björn !
(cherry picked from commit b5011e4c2cee39d4334c04ce7c8adc43a8ca7e6b)

12 years agoPart 5 of bugfix for #8211 - "inherit owner = yes" doesn't interact correctly with...
Jeremy Allison [Wed, 8 Jun 2011 21:37:25 +0000 (14:37 -0700)]
Part 5 of bugfix for #8211 - "inherit owner = yes" doesn't interact correctly with "inherit permissions = yes" and POSIX ACLs

Ensure when creating a directory, if we make any changes due to inheritance parameters, we update the stat returned.
(cherry picked from commit f5e238cbd97d63e107b64268691dff67cce8fe94)

12 years agoPart 4 of bugfix for #8211 - "inherit owner = yes" doesn't interact correctly with...
Jeremy Allison [Wed, 8 Jun 2011 21:21:52 +0000 (14:21 -0700)]
Part 4 of bugfix for #8211 - "inherit owner = yes" doesn't interact correctly with "inherit permissions = yes" and POSIX ACLs

We don't need to check mode bits as well as dev/ino to
ensure we're in the same place.
(cherry picked from commit 0c1b1b73870bd477c83c130cab297b7f2615fe55)

12 years agoPart 3 of bugfix for #8211 - "inherit owner = yes" doesn't interact correctly with...
Jeremy Allison [Wed, 8 Jun 2011 17:25:33 +0000 (10:25 -0700)]
Part 3 of bugfix for #8211 - "inherit owner = yes" doesn't interact correctly with "inherit permissions = yes" and POSIX ACLs

When changing ownership on a new file make sure we
must have a valid stat struct before making the inheritance
calls (as they may look at it), and if we make changes we
must have a valid stat struct after them.
(cherry picked from commit d18d6df840d3a47fa1d7b877e07f804f025811ee)

12 years agoPart 2 of bugfix for #8211 - "inherit owner = yes" doesn't interact correctly with...
Jeremy Allison [Wed, 8 Jun 2011 17:17:42 +0000 (10:17 -0700)]
Part 2 of bugfix for #8211 - "inherit owner = yes" doesn't interact correctly with "inherit permissions = yes" and POSIX ACLs

When changing ownership on a new file make sure we
also change the returned stat struct to have the correct uid.
(cherry picked from commit 59e77811b7774ad76e082ee9fd840a277df75c4c)

12 years agoPart 1 of bugfix for #8211 - "inherit owner = yes" doesn't interact correctly with...
Jeremy Allison [Wed, 8 Jun 2011 17:24:02 +0000 (10:24 -0700)]
Part 1 of bugfix for #8211 - "inherit owner = yes" doesn't interact correctly with "inherit permissions = yes" and POSIX ACLs

When changing ownership on a new directory make sure we
also change the returned stat struct to have the correct uid.
(cherry picked from commit f3900b0a96f98cc65d957cda5f92963f636d6ec1)

12 years agos3:lib/access: normalize IPv4 mapped IPv6 addresses in both directions (bug #7383)
Stefan Metzmacher [Sun, 24 Apr 2011 19:20:19 +0000 (21:20 +0200)]
s3:lib/access: normalize IPv4 mapped IPv6 addresses in both directions (bug #7383)

metze
(cherry picked from commit 4bfe2d5655d97fbc7e65744425b5a098e77f5ba1)
(cherry picked from commit 62b2083c627abeb8a2fb7e5adc793c630d0d561c)

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

12 years agoWHATSNEW: Add more changes since 3.5.8.
Karolin Seeger [Sun, 5 Jun 2011 19:10:53 +0000 (21:10 +0200)]
WHATSNEW: Add more changes since 3.5.8.

Karolin
(cherry picked from commit 7e307ac32ca074e47e27229fcef894343ec0f9c9)

12 years agos3-winbind: BUG 8166 - Don't lockout users when offline.
Jim McDonough [Wed, 25 May 2011 14:49:41 +0000 (10:49 -0400)]
s3-winbind: BUG 8166 - Don't lockout users when offline.

Windows does not track bad password attempts when offline.  We were locking users out but not honoring the lockout duration.

Autobuild-User: Jim McDonough <jmcd@samba.org>
Autobuild-Date: Wed May 25 18:11:10 CEST 2011 on sn-devel-104
(cherry picked from commit b58534f1fca27e3e72f4f4107538ec05734bd42a)
(cherry picked from commit a73963dd49d33bcfdd5cbc310dad0f895683eadf)

12 years agoFix bug #7528 - Solaris with NIS autohome.
Jeremy Allison [Wed, 1 Jun 2011 18:38:48 +0000 (20:38 +0200)]
Fix bug #7528 - Solaris with NIS autohome.
(cherry picked from commit 0ffdf2288b1e6798e43259568818378c43b979e5)

12 years agoWHATSNEW: Start to add changes since 3.5.8.
Karolin Seeger [Tue, 31 May 2011 19:27:29 +0000 (21:27 +0200)]
WHATSNEW: Start to add changes since 3.5.8.

To be completed...

Karolin
(cherry picked from commit ac4aca977ecb5fd1a644aa29c4b70503e906f83a)

12 years agoFix bug #8083 - "inherit owner = yes" doesn't interact correctly with vfs_acl_xattr...
Jeremy Allison [Tue, 19 Apr 2011 20:25:43 +0000 (13:25 -0700)]
Fix bug #8083 - "inherit owner = yes" doesn't interact correctly with vfs_acl_xattr or vfs_acl_tdb module.

If "inherit owner = yes", pass in the directory owner and group
owner as the target for CREATOR_OWNER and CREATOR_GROUP substitutions,
and also as the owner and primary group of the new security descriptor
being applied to the object.

Jeremy.
(cherry picked from commit ea331419108ed8575e33394f989240abeede2671)

12 years agoFix bug #6911 - Kerberos authentication from vista to samba fails when security blob...
Jeremy Allison [Thu, 26 May 2011 23:39:30 +0000 (16:39 -0700)]
Fix bug #6911 - Kerberos authentication from vista to samba fails when security blob size is greater than 16 kB

We were not correctly checking the output of asn1_start_tag().
asn1_start_tag() returns -1 and sets data->has_error if the
remaining blob size is too short to contain the tag length.
We were checking data->has_error and returning NT_STATUS_OK
(to allow the second asn.1 parse to fail in that case). We
should not be checking data->has_error in this case, but
falling through to the code that already checks the length.

Thanks to Jim for reproducing this for me. We don't get bitten
by this as we announce a max buffer size of 16k, greater than
Windows's 4k, which means that most krb5 spnego packets already
fit.

Jeremy.
(cherry picked from commit c718b7d43bc5616f7f5e110afdf5332a62e04cb8)

12 years agos3-libnet: fix bug #6364: Pull realm from supplied username on libnet join
Jim McDonough [Thu, 26 May 2011 18:29:24 +0000 (20:29 +0200)]
s3-libnet: fix bug #6364: Pull realm from supplied username on libnet join
(cherry picked from commit 67a9d4bb376d89c4411024164165ca1ed7a0b157)

12 years agos3-printing: remove duplicate cups response processing code
David Disseldorp [Tue, 24 May 2011 09:50:12 +0000 (11:50 +0200)]
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 4f0077cd6a8ffcb9f4980d71c9fc434f7fc1051c)

12 years agos3-printing: use printcap IDL for IPC
David Disseldorp [Tue, 24 May 2011 09:46:25 +0000 (11:46 +0200)]
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 53ecbc4a8cffe24f94d3e624faf3a9ea9067cc23)

12 years agoidl: define printcap IPC message format
David Disseldorp [Tue, 24 May 2011 09:41:27 +0000 (11:41 +0200)]
idl: define printcap IPC message format

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

12 years agos3-printing: an empty cups printer list is treated as an error
David Disseldorp [Tue, 24 May 2011 09:34:59 +0000 (11:34 +0200)]
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 683b0652e23b787c5038ff5d7466fdc2a8b3c07f)

12 years agoFix our asn.1 parser to handle negative numbers.
Jeremy Allison [Tue, 24 May 2011 19:47:31 +0000 (12:47 -0700)]
Fix our asn.1 parser to handle negative numbers.

Autobuild-User: Jeremy Allison <jra@samba.org>
Autobuild-Date: Tue May 24 22:57:16 CEST 2011 on sn-devel-104
(cherry picked from commit e719dfd4dc178f001a5f804fb1ac4e587574415f)

Fix bug #8163 (asn.1 library does not correctly read negative integers).
(cherry picked from commit 859d13141cd831488b60e413f7141514ae4464b5)
(cherry picked from commit d210395a50b5d5043bdcfb75f670f8abab91f974)

12 years agoFix bug #8157 - std_pcap_cache_reload() fails to parse a cups printcap file correctly.
Jeremy Allison [Fri, 20 May 2011 19:36:56 +0000 (12:36 -0700)]
Fix bug #8157 - std_pcap_cache_reload() fails to parse a cups printcap file correctly.

The parsing code made some strange assumptions about what is a printer
name, and what is a comment.
(cherry picked from commit ef1a0c14ab41c87f133d310c5f976548caf15b9a)

12 years agos3-printing: remove pcap_cache_loaded asserts
David Disseldorp [Mon, 17 Jan 2011 15:09:32 +0000 (16:09 +0100)]
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 a88126d6e8577a9e0b6196acdee70633d0e06259)

12 years agoRevert "Revert "s3-printing: update parent smbd pcap cache""
David Disseldorp [Thu, 19 May 2011 09:22:37 +0000 (11:22 +0200)]
Revert "Revert "s3-printing: update parent smbd pcap cache""

This reverts commit b6268f507fa3276c2ef22c58bad400a3fed48cd9.

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

12 years agoRevert "Revert "s3-printing: reload shares after pcap cache fill""
David Disseldorp [Thu, 19 May 2011 09:22:18 +0000 (11:22 +0200)]
Revert "Revert "s3-printing: reload shares after pcap cache fill""

This reverts commit e4579eab7fe3eab7a5209e6de74e6fd2f53099d0.

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

12 years agoRevert "s3-spoolss: Added EN ISO 216, A0 and A1 to builtin forms."
Karolin Seeger [Mon, 23 May 2011 18:17:13 +0000 (20:17 +0200)]
Revert "s3-spoolss: Added EN ISO 216, A0 and A1 to builtin forms."

This reverts commit 73bec197a91a15aa9a69c9a3868ed51bdd3674ea.

Please see bug #8129 (Application requests printing on Format A5, but prints as
A4) fro details.
(cherry picked from commit f51ee94ee9034f88566c4441eed4ff11697454ec)

12 years agolibwbclient: Fix bug 8087 -- wbcChangeUserPasswordEx in RESPONSE mode does not work
Volker Lendecke [Fri, 23 Apr 2010 17:41:29 +0000 (19:41 +0200)]
libwbclient: Fix bug 8087 -- wbcChangeUserPasswordEx in RESPONSE mode does not work

This is 03115efae89c8c4f51dea1ce82613817bd9fcf5b from master

Actually copy something in wbcChangeUserPasswordEx

The length argument for memcpy was initialized to 0 and not initialized
(cherry picked from commit c707b1f3b199b8c785a79db308d80eee2926b060)

12 years agos3-net: make sure we dont crash when publishing a single printer.
Günther Deschner [Fri, 1 Oct 2010 04:08:12 +0000 (06:08 +0200)]
s3-net: make sure we dont crash when publishing a single printer.

Guenther
(cherry picked from commit 21576e3f8c32878910460bf9575c200ad93d682a)

Part of a fix for bug #7993 ("net rpc printer MIGRATE" command fails).
(cherry picked from commit 9c75728c62cccb8da606ece2d9df08b592e7c7c0)

12 years agos3/configure: fix GNU ld version detection with old gcc releases
Björn Jacke [Fri, 26 Nov 2010 14:14:14 +0000 (15:14 +0100)]
s3/configure: fix GNU ld version detection with old gcc releases

needed as old gcc releases output everything to stderr, even stdout output from
ld

Fixes #7825

Autobuild-User: Björn Jacke <bj@sernet.de>
Autobuild-Date: Fri Nov 26 20:15:24 CET 2010 on sn-devel-104
(cherry picked from commit 70a7da0e101910e3ceb08b86d4b840b219e24d7d)
(cherry picked from commit 19bbd0a4f0c39aaf6f0f3ecdcce7c52bb8264a5a)

12 years agotdb_expand: limit the expansion with huge records
Simo Sorce [Mon, 18 Apr 2011 12:45:11 +0000 (22:15 +0930)]
tdb_expand: limit the expansion with huge records

ldb can create huge records when saving indexes.
Limit the tdb expansion to avoid consuming a lot of memory for
no good reason if the record being saved is huge.

Fix bug #7610 (winbindd_cache.tdb grows too large when scaled).
(cherry picked from commit c8ba5d41f3c2ab25cb9b9d0fa78b4f884d4b9721)

12 years agos3-printing: make cups_pull_comment_location() work again.
Günther Deschner [Wed, 11 May 2011 08:30:42 +0000 (10:30 +0200)]
s3-printing: make cups_pull_comment_location() work again.

we deal with lp_cups_server in cups_connect() already, inside the URI all our
other cups functions we use ipp://localhost, do the same here.

Guenther

Autobuild-User: Günther Deschner <gd@samba.org>
Autobuild-Date: Wed May 11 11:36:07 CEST 2011 on sn-devel-104

The last 3 patches address bug #8132 (Samba does not fill printers Location
field when using cups).
(cherry picked from commit 392d6b2ab8dcde4176a6e872699a5a076ab92068)

12 years agos3-printing: Fix double free of cups request.
Günther Deschner [Tue, 10 May 2011 13:49:05 +0000 (15:49 +0200)]
s3-printing: Fix double free of cups request.

We never free the request in our cups api usage except for here. The reason is
probably htis (from the cupsDoConnect API docs):

"This function sends the IPP request to the specified server, retrying and
authenticating as necessary. The request is freed with ippDelete() after
receiving a valid IPP response."

Revert "Fix a memory leak in cups_pull_comment_location"

This reverts commit fee2664dad37536b05ce8bdae3e74d45b257f632.

Autobuild-User: Günther Deschner <gd@samba.org>
Autobuild-Date: Tue May 10 17:32:58 CEST 2011 on sn-devel-104
(cherry picked from commit 019f11dd5b3240d05c1abe30dec3d793d6919313)
(cherry picked from commit 5d503c7e8927f2bdb252f2293e53b7e87f2d3cb5)

12 years agos3-printing: very obvious fix for cups_pull_comment_location().
Günther Deschner [Tue, 10 May 2011 13:48:25 +0000 (15:48 +0200)]
s3-printing: very obvious fix for cups_pull_comment_location().

This has been in there since 2008...

Guenther
(cherry picked from commit 3ba3f68e03510e3bb5b7627c200af0395e853bc2)
(cherry picked from commit 04e820ef17e5a9df90ff8b7744bf9cce5a00ae05)

12 years agos3-libsmb Don't ever ask for machine$ principals as a target.
Andrew Bartlett [Thu, 9 Dec 2010 20:57:59 +0000 (07:57 +1100)]
s3-libsmb Don't ever ask for machine$ principals as a target.

It is never correct to ask for a machine$ principal as the target of a
kerberos connection.  You should always connect via the
servicePrincipalName.

This current code appears to have built up from a series of minimal
changes, as the codebase adapted the to lack of a SPNEGO principal
from Windows 2008.

Andrew Bartlett

The last two patches address bug #7893 (CIFS tickets vs. <host>$ tickets).
(cherry picked from commit 12bb20a0f6cd85cfdaedf746e7b05416ccde31b0)

12 years agos3-libads Default to NOT using the server-supplied principal from SPNEGO
Andrew Bartlett [Sat, 4 Dec 2010 02:48:37 +0000 (13:48 +1100)]
s3-libads Default to NOT using the server-supplied principal from SPNEGO

This principal is not supplied by later versions of windows, and using
it opens up some oportunities for man in the middle attacks.  (Becuase
it isn't the name being contacted that is verified with the KDC).

This adds the option 'client use spnego principal' to the smb.conf (as
used in Samba4) to control this behaivour.  As in Samba4, this
defaults to false.

Against 2008 servers, this will not change behaviour.  Against earlier
servers, it may cause a downgrade to NTLMSSP more often, in
environments where server names are not registered with the KDC as
servicePrincipalName values.

Andrew Bartlett
(cherry picked from commit bb7806283e71f3b8029aae0eed326b5847a36d83)
(cherry picked from commit e962852687f539678b7c38ed21f1b76c328821f2)

12 years agoFix bug 8111 - CIFS VFS: unexpected error on SMB posix open
Jeremy Allison [Fri, 29 Apr 2011 21:22:54 +0000 (14:22 -0700)]
Fix bug 8111 - CIFS VFS: unexpected error on SMB posix open

We are conflating the O_CREAT|O_EXCL with the O_TRUNC
processing, they need to be separate. We need to chose
using (O_CREAT|O_EXCL) first, then modify if O_TRUNC is
set. This needs two separate switch statements.

Jeremy
(cherry picked from commit 37823155157d735356e1f223b425252c956d8c04)

12 years agoFix is_myname_or_ipaddr() to be robust against strange DNS setups.
Jeremy Allison [Thu, 24 Mar 2011 18:55:38 +0000 (11:55 -0700)]
Fix is_myname_or_ipaddr() to be robust against strange DNS setups.

If IPv6 DNS names are turned on, but Samba isn't configured to
listen on an IPv6 interface, then is_myname_or_ipaddr() can return
false on a valid DNS name that it should detect is our own. If the
IPv6 addr is returned by preference, then looking at the first addr
only causes is_myname_or_ipaddr() to fail. We need to look at all the
addresses returned by the DNS lookup and check all of them against
our interface list. This is an order N^2 lookup, but there shouldn't
be enough addresses to make this a practical problem.

Jeremy.

Fix bug #8038 - Connecting to a printer can return INVALID_PARAMETER when IPv6
DNS names are turned on.
(cherry picked from commit 80078cb6ef2e6976cb5ab25a86157bca22c836a2)

12 years agos3: Fix bug 8099 - setpwent() actually does endpwent() on FreeBSD
Sergey Korsak [Tue, 19 Apr 2011 16:51:32 +0000 (18:51 +0200)]
s3: Fix bug 8099 - setpwent() actually does endpwent() on FreeBSD
(cherry picked from commit 2167ac2cd42c9ed5aaae0086dbd27e29d1d77686)

12 years agoFix bug 8072 - PANIC: create_file_acl_common frees handle two times.
Jeremy Allison [Fri, 8 Apr 2011 22:25:18 +0000 (15:25 -0700)]
Fix bug 8072 - PANIC: create_file_acl_common frees handle two times.

Caused by premature optimisation storing the parent ACL on the
module handle instead of (correctly) on the file fsp. Previous
code wasn't reentrant safe. This is less optimal but doesn't
crash in the specific case :-).

Jeremy.
(cherry picked from commit 23e6f41ec923e2d3b4684ee646c8cd29506d787a)

12 years agoFix bug 8088 - rpccli_samr_chng_pswd_auth_crap segfaults if any input blobs are null.
Jeremy Allison [Mon, 18 Apr 2011 21:26:09 +0000 (14:26 -0700)]
Fix bug 8088 - rpccli_samr_chng_pswd_auth_crap segfaults if any input blobs are null.
(cherry picked from commit fae43d2640459fe8cb3d485eacd1624de59b9622)

12 years agoFix bug 6966 - "allow trusted domains = no" not respected in winbind.
Dmitry Butskoy [Mon, 18 Apr 2011 21:14:09 +0000 (14:14 -0700)]
Fix bug 6966 - "allow trusted domains = no" not respected in winbind.
(cherry picked from commit 66de22f7b242ae9e3358d33e580547c2618bd918)

12 years agos3: Fix bug 8066, wrong output in smbget
Volker Lendecke [Thu, 7 Apr 2011 20:03:49 +0000 (22:03 +0200)]
s3: Fix bug 8066, wrong output in smbget
(cherry picked from commit 4a467155fe9ad66a2d7e7e490d7e3957b4dcb67a)

12 years agoFix bug #7987 - ACL can get lost when files are being renamed.
Jeremy Allison [Wed, 6 Apr 2011 00:26:00 +0000 (17:26 -0700)]
Fix bug #7987 - ACL can get lost when files are being renamed.

There is no reason for smbd with Windows ACLs to use chmod
or fchmod unless it's a file opened with UNIX extensions or
with posix pathnames.
(cherry picked from commit bea18a55252a06fd1da005a4f57d4d4dd89acaaf)

12 years agos3-cli_pipe: fix timeout in rpc_pipe_open_tcp_port().
Günther Deschner [Wed, 13 Apr 2011 15:41:36 +0000 (17:41 +0200)]
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)

12 years agos3-docs: document all wbinfo options.
Günther Deschner [Tue, 12 Apr 2011 08:22:23 +0000 (10:22 +0200)]
s3-docs: document all wbinfo options.

Guenther

Autobuild-User: Günther Deschner <gd@samba.org>
Autobuild-Date: Wed Feb 23 23:45:40 CET 2011 on sn-devel-104

Fix bug #7983 - not all wbinfo parameters are documented in manpage.
(cherry picked from commit 7942bf50ce22597833bbfba99776f4b052edff1c)

12 years agos3-modules: Fix debug message (bug #8074)
Björn Baumbach [Mon, 11 Apr 2011 08:27:58 +0000 (10:27 +0200)]
s3-modules: Fix debug message (bug #8074)

Print child descriptor instead of parent.

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Autobuild-User: Stefan Metzmacher <metze@samba.org>
Autobuild-Date: Mon Apr 11 11:48:42 CEST 2011 on sn-devel-104
(cherry picked from commit e6cf92c574fba14132757f141d8b1242fa71be88)
(cherry picked from commit 42ad7630259829f1c40d9d0fcf5376fa007568a3)
(cherry picked from commit 75ab0c486927f674937b3379104eb2e4c8f026e4)

12 years agoFix bug #6762 - ctdb on gpfs error with MS Office.
Martin Vogt [Thu, 7 Apr 2011 19:20:06 +0000 (21:20 +0200)]
Fix bug #6762 - ctdb on gpfs error with MS Office.
(cherry picked from commit 84745fe2ef953b1b5edfef473c88c71cc71d4d1e)

12 years agos3/vfs_gpfs: s/syncops/gpfs
Björn Jacke [Sun, 3 Apr 2011 14:19:11 +0000 (16:19 +0200)]
s3/vfs_gpfs: s/syncops/gpfs

as pointed out by Metze in bug #8031

cherry-picked from dca465fa53f4d16cdce1353685b11010aa8ff0c7

The last two patches address bug #8031 - merge patc to make
sharemodes/leases parameter a per share setting.
(cherry picked from commit fbf1a26be29f3d78d09e5f4285c973db9e16327f)

12 years agos3:vfs:gpfs convert sharemodes/leases parameter
Christian Ambach [Fri, 8 Oct 2010 11:15:57 +0000 (13:15 +0200)]
s3:vfs:gpfs convert sharemodes/leases parameter

convert gpfs:sharemodes and gpfs:leases parameters from a global setting
to a per share setting

cherry-picked from 22018b8b887c2677d30bbb4589f800197edf0e98
(cherry picked from commit 4413d05bd742f879a6af71206265791ff76070bb)

12 years agos3: Fix bug 8042: File creation on OS/X
Volker Lendecke [Mon, 4 Apr 2011 17:19:18 +0000 (10:19 -0700)]
s3: Fix bug 8042: File creation on OS/X

With a case insensitive file system the stat cache lookup leaked the parent
directorys stat information from unix_convert into the smb_filename. This led
open_file_ntcreate to believe it just created a directory.

In the case where we do the search we already invalidate the stat struct.

Thanks to TAKAHASHI Motonobu for insisting! :-)

Volker
(cherry picked from commit c9015e381905bb254ee61b64d99052b96b4d4913)

12 years agoFix bug #7080 - Quota only shown when logged as root.
Jeremy Allison [Tue, 5 Apr 2011 21:15:56 +0000 (14:15 -0700)]
Fix bug #7080 - Quota only shown when logged as root.

Ensure we also check conn->admin_user when rejecting non-root access.
(cherry picked from commit 6d360c9e5db64fbd96d353013b7836d771935b6d)

12 years agoFix for servers that don't put a path separator at the end of the service.
Larry Reid [Sat, 26 Mar 2011 22:39:27 +0000 (15:39 -0700)]
Fix for servers that don't put a path separator at the end of the service.

Fix bug 8055 - Can't See Parts of DFS CIFS Share.
(cherry picked from commit 0d929df7d194574d283ea5b9f4f8a45d6d214a54)

12 years agos3: Fix Coverity ID 1137: CONSTANT_EXPRESSION_RESULT
Volker Lendecke [Fri, 1 Apr 2011 06:40:38 +0000 (08:40 +0200)]
s3: Fix Coverity ID 1137: CONSTANT_EXPRESSION_RESULT

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Fri Apr  1 09:35:19 CEST 2011 on sn-devel-104

This patch and the last one address bug #8054 - winbindd cache stores/retrieves
wrong sizes for 16-bit ints.
(cherry picked from commit 5fc7b78c64c43fd345b96dcbc1d30f2cabfaef81)

12 years agos3: Fix Coverity ID 1136: CONSTANT_EXPRESSION_RESULT
Volker Lendecke [Fri, 1 Apr 2011 06:40:38 +0000 (08:40 +0200)]
s3: Fix Coverity ID 1136: CONSTANT_EXPRESSION_RESULT
(cherry picked from commit ec4d201d5b9b86dfead5d999e4f186316797f77f)

12 years agos3: Fix Bug 8047 -- Fix mdns registration if "interfaces=" is used
Marc A. Dahlhaus [Fri, 1 Apr 2011 20:24:56 +0000 (22:24 +0200)]
s3: Fix Bug 8047 -- Fix mdns registration if "interfaces=" is used
(cherry picked from commit baecb5a35e9f50140cf78e2dbdfe9f2791653875)

12 years agoalpha_strcpy() is a utility function which reportedly: Strips out all but 'a-Z0-9...
David Disseldorp [Fri, 1 Apr 2011 18:21:59 +0000 (11:21 -0700)]
alpha_strcpy() is a utility function which reportedly: Strips out all but 'a-Z0-9' and the character in other_safe_chars and replaces with '_'.

This statement does not currently hold true in all cases (e.g. src =
"ТАНЦЕВАТЬ").

Part of a fix for bug 8040 - smbclient segfaults when a Cyrillic netbios
name or workgroup is configured.
(cherry picked from commit 3e0f539596fbb867b672eeaff037e81c33428309)

12 years agoFix bug 8040 - smbclient segfaults when a Cyrillic netbios name or workgroup is confi...
Jeremy Allison [Fri, 25 Mar 2011 22:12:12 +0000 (15:12 -0700)]
Fix bug 8040 - smbclient segfaults when a Cyrillic netbios name or workgroup is configured.

As discovered by David Disseldorp <ddiss@suse.de>, convert_string_talloc()
doesn't always return consistent results for a zero length string. The
API states an incoming string must *always* contain the terminating null,
but unfotunately too much code expects passing in a zero source length
to return a null terminated string, so at least ensure we return a
correct null string in the required character set and return the
correct length.

Also ensure we cannot return a zero length for a converted string
(we ensure that the returned buffer is always allocated and zero
terminated anyway) as calling code depends on the fact that returning
true from this function will *always* return a non-zero length (as
it must include the terminating null).

Note this is a different fix from what went into master (this is
identical to the fix I'm planning for 3.5.x) as convert_string_talloc()
has diverged between the two.

Jeremy.
(cherry picked from commit bb3ed43584e6d2c4d64b5f7b9e70a7db7f3e859d)

12 years agoFix bug #7996 - sgid bit lost on folder rename.
Jeremy Allison [Thu, 31 Mar 2011 17:49:22 +0000 (10:49 -0700)]
Fix bug #7996 - sgid bit lost on folder rename.

Refuse to set dos attributes into unix mode bits on such a
folder.
(cherry picked from commit 90e7f310ec52119359784899945f47d0a9c4e3ae)

12 years agos3: use getgrset() when it is available
Christian Ambach [Mon, 14 Mar 2011 12:08:58 +0000 (08:08 -0400)]
s3: use getgrset() when it is available

When getgrouplist() is not defined, use getgrset() if it is defined
instead of using the initgroups() + getgroups() combo

Major contributions from Yannick Bergeron <yaberger@ca.ibm.com>

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Sat Mar 19 10:09:38 CET 2011 on sn-devel-104
(cherry picked from commit ed46dfc4f16e230645fae5f3b3b21c462694c30a)

Fix bug #8012 (Use getgrset() instead of initgroups() + getgroups() when
getgrouplist() is not defined).
(cherry picked from commit 64be11d41292fd2e9f6c13855fa6041b9290ce0c)

12 years agos3: add explicit configure option whether or not to enable dmapi support
Björn Jacke [Tue, 23 Feb 2010 14:23:27 +0000 (15:23 +0100)]
s3: add explicit configure option whether or not to enable dmapi support

Signed-off-by: Stefan Metzmacher <metze@samba.org>
(similar to commit ce7727c1cc2fe4caa9b2d6e33530c3483dd5f980)

Fix bug #8033 - Backport dmapi configure checks.
(cherry picked from commit a8552d4ea6ea3b3b59b0d1c39e8fdaf2f9e11a74)

12 years agos3:util_seaccess: make sure that we don't grant SEC_STD_DELETE to the owner by default
Stefan Metzmacher [Mon, 21 Mar 2011 13:15:32 +0000 (14:15 +0100)]
s3:util_seaccess: make sure that we don't grant SEC_STD_DELETE to the owner by default

In the file server SEC_STD_DELETE is granted on the file/directory
or by FILE_DELETE_CHILD on the parent directory.

metze
(similar to commit c7d10179108a3ae8af15c838042294f3fdced03c)

The last 2 patches address bug #8034 (SEC_STD_DELETE is always granted to the
owner of a file).
(cherry picked from commit 14a31111961278db99564d4d694f10ed66ff91bd)

12 years agos3:smbd: access checks should not depend on share mode flags
Stefan Metzmacher [Fri, 18 Mar 2011 15:45:08 +0000 (16:45 +0100)]
s3:smbd: access checks should not depend on share mode flags

metze
(cherry picked from commit f0ec69b53544b7ff702f94d58b3d64c33eaabc7a)
(cherry picked from commit dca75c8e3ac23256d93269bdc13d1e67571bc9f2)
(cherry picked from commit 295271ca3d4346e130363ba227ff66b08368d957)