The branch, v4-2-test has been updated via b065b1e s3: smbd: Fix timestamp rounding inside SMB2 create. via 7205d15 s3:utils/smbget fix recursive download via da8f785 waf: Only build smb_krb5_wrapper if we have CUPS via 542cbdf s3-client: Add a KRB5 wrapper for smbspool from de7ad5d VERSION: Bump version up to 4.2.9...
https://git.samba.org/?p=samba.git;a=shortlog;h=v4-2-test - Log ----------------------------------------------------------------- commit b065b1e1ebd5d0b7a0428a324c01a715febaedc8 Author: Jeremy Allison <j...@samba.org> Date: Thu Jan 28 16:35:13 2016 -0800 s3: smbd: Fix timestamp rounding inside SMB2 create. Setting "dos filetime resolution = yes" should round to 2 second increments. Fix missing path inside smbd. Fix confirmed by reporter Hubert Gilch <h...@sepag.de>. BUG: https://bugzilla.samba.org/show_bug.cgi?id=11703 Signed-off-by: Jeremy Allison <j...@samba.org> Reviewed-by: Volker Lendecke <v...@samba.org> Autobuild-User(master): Volker Lendecke <v...@samba.org> Autobuild-Date(master): Fri Jan 29 10:41:01 CET 2016 on sn-devel-144 (cherry picked from commit 8024f532e6bcb6cbd8179817fafdd0f12929d629) Autobuild-User(v4-2-test): Karolin Seeger <ksee...@samba.org> Autobuild-Date(v4-2-test): Thu Feb 4 11:28:24 CET 2016 on sn-devel-104 commit 7205d15a135d28926ec8cee0165989b7478f8705 Author: Christian Ambach <a...@samba.org> Date: Wed Dec 30 21:25:13 2015 +0100 s3:utils/smbget fix recursive download get_auth_data is called multiple times (once for the directory listing and then for every file to be downloaded). Save the obtained values across multiple calls to make smbclient use the correct username for each download. Bug: https://bugzilla.samba.org/show_bug.cgi?id=6482 Signed-off-by: Christian Ambach <a...@samba.org> Reviewed-by: Andreas Schneider <a...@samba.org> (cherry picked from commit ec802d27ce4dc6dd9b5e5ebd6992f90364d855a2) commit da8f7858dd99d8add004d6aa5168ae85749c26f0 Author: Andreas Schneider <a...@samba.org> Date: Tue Jan 26 11:28:50 2016 +0100 waf: Only build smb_krb5_wrapper if we have CUPS Signed-off-by: Andreas Schneider <a...@samba.org> Reviewed-by: Michael Adam <ob...@samba.org> Reviewed-by: Stefan Metzmacher <me...@samba.org> (cherry picked from commit 192f1516c378ae942d14921bfcc1e11173da36e6) commit 542cbdfa03cabf2e061e6e9ec0f3b2b7e3d92fb6 Author: Andreas Schneider <a...@cryptomilk.org> Date: Tue Jan 12 15:17:22 2016 +0100 s3-client: Add a KRB5 wrapper for smbspool BUG: https://bugzilla.samba.org/show_bug.cgi?id=11690 We need a wrapper for smbspool to be able to authenticate with Kerberos. This needs to replace the cups smb backend. The permission need to be 0700 and the owner root. Note that Kerberos support is broken in CUPS 2.1.2 maybe earlier versions. It works with 1.6.3. Signed-off-by: Andreas Schneider <a...@cryptomilk.org> Reviewed-by: Michael Adam <ob...@samba.org> Reviewed-by: Guenther Deschner <g...@samba.org> Reviewed-by: Ralph Boehme <s...@samba.org> Autobuild-User(master): Andreas Schneider <a...@cryptomilk.org> Autobuild-Date(master): Mon Jan 25 19:58:46 CET 2016 on sn-devel-144 (cherry picked from commit 62c68bd8c8f7a7e057e38a461707f1c195e62df0) ----------------------------------------------------------------------- Summary of changes: source3/client/README.smbspool | 17 +++ source3/client/smbspool_krb5_wrapper.c | 210 +++++++++++++++++++++++++++++++++ source3/smbd/smb2_create.c | 8 ++ source3/utils/smbget.c | 15 ++- source3/wscript_build | 8 ++ 5 files changed, 257 insertions(+), 1 deletion(-) create mode 100644 source3/client/README.smbspool create mode 100644 source3/client/smbspool_krb5_wrapper.c Changeset truncated at 500 lines: diff --git a/source3/client/README.smbspool b/source3/client/README.smbspool new file mode 100644 index 0000000..f73167a6 --- /dev/null +++ b/source3/client/README.smbspool @@ -0,0 +1,17 @@ +smbspool +========= + +smbspool is a very small print spooling program that sends a print file to an +SMB printer. The command-line arguments are position-dependent for +compatibility with the CUPS. + +For printing support with Kerberos, CUPS 1.5+ needs a wrapper for the backend +which sets the correct location of the Kerberos credential cache. + +smbspool_krb5_wrapper +====================== + +This tool can be used to print using Kerberos credentials. To get this working +smbspool_krb5_wrapper needs to be the smb backend of CUPS. It needs to be owned +by root and the permissions for the binary need to be 0700. Once +smbspool_krb5_wrapper switched to the user trying to print it executes smbspool. diff --git a/source3/client/smbspool_krb5_wrapper.c b/source3/client/smbspool_krb5_wrapper.c new file mode 100644 index 0000000..e19fd92 --- /dev/null +++ b/source3/client/smbspool_krb5_wrapper.c @@ -0,0 +1,210 @@ +/* + * Unix SMB/CIFS implementation. + * + * CUPS printing backend helper to execute smbspool + * + * Copyright (C) 2010-2011 Andreas Schneider <a...@samba.org> + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License as published by + * the Free Software Foundation; either version 3 of the License, or + * (at your option) any later version. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + * + * You should have received a copy of the GNU General Public License + * along with this program. If not, see <http://www.gnu.org/licenses/>. + */ + +#include "includes.h" +#include "system/filesys.h" +#include "system/passwd.h" + +#include <errno.h> +#include <string.h> + +#include <cups/backend.h> + +#include "dynconfig/dynconfig.h" + +enum cups_smb_dbglvl_e { + CUPS_SMB_LOG_DEBUG = 0, + CUPS_SMB_LOG_ERROR, +}; +static void cups_smb_debug(enum cups_smb_dbglvl_e lvl, const char *format, ...); + +#define CUPS_SMB_DEBUG(...) cups_smb_debug(CUPS_SMB_LOG_DEBUG, __VA_ARGS__) +#define CUPS_SMB_ERROR(...) cups_smb_debug(CUPS_SMB_LOG_DEBUG, __VA_ARGS__) + +static void cups_smb_debug(enum cups_smb_dbglvl_e lvl, const char *format, ...) +{ + const char *prefix = "DEBUG"; + char buffer[1024]; + va_list va; + + va_start(va, format); + vsnprintf(buffer, sizeof(buffer), format, va); + va_end(va); + + switch (lvl) { + case CUPS_SMB_LOG_DEBUG: + prefix = "DEBUG"; + break; + case CUPS_SMB_LOG_ERROR: + prefix = "ERROR"; + break; + } + + fprintf(stderr, + "%s: SMBSPOOL_KRB5 - %s\n", + prefix, + buffer); +} + +/* + * This is a helper binary to execute smbspool. + * + * It needs to be installed or symlinked as: + * /usr/lib/cups/backend/smb + * + * The permissions of the binary need to be set to 0700 so that it is executed + * as root. The binary switches to the user which is passed via the environment + * variable AUTH_UID, so we can access the kerberos ticket. + */ +int main(int argc, char *argv[]) +{ + char smbspool_cmd[PATH_MAX] = {0}; + struct passwd *pwd; + char gen_cc[PATH_MAX] = {0}; + struct stat sb; + char *env; + uid_t uid = (uid_t)-1; + gid_t gid = (gid_t)-1; + unsigned long tmp; + int cmp; + int rc; + + uid = getuid(); + + CUPS_SMB_DEBUG("Started with uid=%d\n", uid); + if (uid != 0) { + goto smbspool; + } + + /* Check if AuthInfoRequired is set to negotiate */ + env = getenv("AUTH_INFO_REQUIRED"); + if (env == NULL) { + CUPS_SMB_ERROR("AUTH_INFO_REQUIRED is not set"); + fprintf(stderr, "ATTR: auth-info-required=negotiate\n"); + return CUPS_BACKEND_AUTH_REQUIRED; + } + + CUPS_SMB_DEBUG("AUTH_INFO_REQUIRED=%s", env); + cmp = strcmp(env, "negotiate"); + if (cmp != 0) { + CUPS_SMB_ERROR("AUTH_INFO_REQUIRED is not set to negotiate"); + fprintf(stderr, "ATTR: auth-info-required=negotiate\n"); + return CUPS_BACKEND_AUTH_REQUIRED; + } + + /* + * AUTH_UID gets only set if we have an incoming connection over the + * CUPS unix domain socket. + */ + env = getenv("AUTH_UID"); + if (env == NULL) { + CUPS_SMB_ERROR("AUTH_UID is not set"); + fprintf(stderr, "ATTR: auth-info-required=negotiate\n"); + return CUPS_BACKEND_AUTH_REQUIRED; + } + + if (strlen(env) > 10) { + CUPS_SMB_ERROR("Invalid AUTH_UID"); + return CUPS_BACKEND_FAILED; + } + + errno = 0; + tmp = strtoul(env, NULL, 10); + if (errno != 0 || tmp >= UINT32_MAX) { + CUPS_SMB_ERROR("Failed to convert AUTH_UID=%s", env); + return CUPS_BACKEND_FAILED; + } + uid = (uid_t)tmp; + + pwd = getpwuid(uid); + if (pwd == NULL) { + CUPS_SMB_ERROR("Failed to find system user: %u - %s", + uid, strerror(errno)); + return CUPS_BACKEND_FAILED; + } + gid = pwd->pw_gid; + + rc = setgroups(0, NULL); + if (rc != 0) { + CUPS_SMB_ERROR("Failed to clear groups - %s", + strerror(errno)); + return CUPS_BACKEND_FAILED; + } + + CUPS_SMB_DEBUG("Switching to gid=%d", gid); + rc = setgid(gid); + if (rc != 0) { + CUPS_SMB_ERROR("Failed to switch to gid=%u", + gid, + strerror(errno)); + return CUPS_BACKEND_FAILED; + } + + CUPS_SMB_DEBUG("Switching to uid=%u", uid); + rc = setuid(uid); + if (rc != 0) { + CUPS_SMB_ERROR("Failed to switch to uid=%u", + uid, + strerror(errno)); + return CUPS_BACKEND_FAILED; + } + + snprintf(gen_cc, sizeof(gen_cc), "/tmp/krb5cc_%d", uid); + + rc = lstat(gen_cc, &sb); + if (rc == 0) { + snprintf(gen_cc, sizeof(gen_cc), "FILE:/tmp/krb5cc_%d", uid); + } else { + snprintf(gen_cc, sizeof(gen_cc), "/run/user/%d/krb5cc", uid); + + rc = lstat(gen_cc, &sb); + if (rc == 0 && S_ISDIR(sb.st_mode)) { + snprintf(gen_cc, + sizeof(gen_cc), + "DIR:/run/user/%d/krb5cc", + uid); + } else { +#if defined(__linux__) + snprintf(gen_cc, + sizeof(gen_cc), + "KEYRING:persistent:%d", + uid); +#endif + } + } + + /* + * Make sure we do not have LD_PRELOAD or other security relevant + * environment variables set. + */ + clearenv(); + + CUPS_SMB_DEBUG("Setting KRB5CCNAME to '%s'", gen_cc); + setenv("KRB5CCNAME", gen_cc, 1); + +smbspool: + snprintf(smbspool_cmd, + sizeof(smbspool_cmd), + "%s/smbspool", + get_dyn_BINDIR()); + + return execv(smbspool_cmd, argv); +} diff --git a/source3/smbd/smb2_create.c b/source3/smbd/smb2_create.c index ecedd08..2112395 100644 --- a/source3/smbd/smb2_create.c +++ b/source3/smbd/smb2_create.c @@ -1265,6 +1265,14 @@ static struct tevent_req *smbd_smb2_create_send(TALLOC_CTX *mem_ctx, state->out_last_write_ts = result->fsp_name->st.st_ex_mtime; state->out_change_ts = get_change_timespec(smb1req->conn, result, result->fsp_name); + + if (lp_dos_filetime_resolution(SNUM(smb2req->tcon->compat))) { + dos_filetime_timespec(&state->out_creation_ts); + dos_filetime_timespec(&state->out_last_access_ts); + dos_filetime_timespec(&state->out_last_write_ts); + dos_filetime_timespec(&state->out_change_ts); + } + state->out_allocation_size = SMB_VFS_GET_ALLOC_SIZE(smb1req->conn, result, &(result->fsp_name->st)); diff --git a/source3/utils/smbget.c b/source3/utils/smbget.c index 28d5647..2c7dfc3 100644 --- a/source3/utils/smbget.c +++ b/source3/utils/smbget.c @@ -91,10 +91,18 @@ static void human_readable(off_t s, char *buffer, int l) static void get_auth_data(const char *srv, const char *shr, char *wg, int wglen, char *un, int unlen, char *pw, int pwlen) { static char hasasked = 0; + static char *savedwg; + static char *savedun; + static char *savedpw; char *wgtmp, *usertmp; char tmp[128]; - if(hasasked) return; + if (hasasked) { + strncpy(wg, savedwg, wglen - 1); + strncpy(un, savedun, unlen - 1); + strncpy(pw, savedpw, pwlen - 1); + return; + } hasasked = 1; if(!nonprompt && !username) { @@ -119,6 +127,11 @@ static void get_auth_data(const char *srv, const char *shr, char *wg, int wglen, if(workgroup)strncpy(wg, workgroup, wglen-1); + /* save the values found for later */ + savedwg = SMB_STRDUP(wg); + savedun = SMB_STRDUP(un); + savedpw = SMB_STRDUP(pw); + wgtmp = SMB_STRNDUP(wg, wglen); usertmp = SMB_STRNDUP(un, unlen); if(!quiet)printf("Using workgroup %s, %s%s\n", wgtmp, *usertmp?"user ":"guest user", usertmp); diff --git a/source3/wscript_build b/source3/wscript_build index 27f3783..58e1b99 100755 --- a/source3/wscript_build +++ b/source3/wscript_build @@ -1152,6 +1152,14 @@ bld.SAMBA3_BINARY('smbspool', libsmb samba3core''') +bld.SAMBA3_BINARY('smbspool_krb5_wrapper', + source='client/smbspool_krb5_wrapper.c', + deps=''' + DYNCONFIG + cups + ''', + enabled=bld.CONFIG_SET('HAVE_CUPS')) + bld.SAMBA3_BINARY('testparm', source='utils/testparm.c', deps=''' -- Samba Shared Repository