Acked-by: Igor Mammedov <niallain@xxxxxxxxx>
On Fri, Jan 7, 2011 at 5:11 PM, Jeff Layton <jlayton@xxxxxxxxx> wrote:
> Signed-off-by: Jeff Layton <jlayton@xxxxxxxxx>
> ---
> Âcifs.upcall.c | Â 28 +++++++++++++---------------
> Â1 files changed, 13 insertions(+), 15 deletions(-)
>
> diff --git a/cifs.upcall.c b/cifs.upcall.c
> index 34b0638..33b7e4c 100644
> --- a/cifs.upcall.c
> +++ b/cifs.upcall.c
> @@ -675,6 +675,7 @@ int main(const int argc, char *const argv[])
> Â Â Â Âchar hostbuf[NI_MAXHOST], *host;
> Â Â Â Âstruct decoded_args arg;
> Â Â Â Âconst char *oid;
> + Â Â Â uid_t uid;
>
> Â Â Â Âhostbuf[0] = '\0';
> Â Â Â Âmemset(&arg, 0, sizeof(arg));
> @@ -748,27 +749,24 @@ int main(const int argc, char *const argv[])
> Â Â Â Â Â Â Â Âgoto out;
> Â Â Â Â}
>
> - Â Â Â if (!legacy_uid && (have & DKD_HAVE_CREDUID)) {
> - Â Â Â Â Â Â Â rc = setuid(arg.creduid);
> - Â Â Â Â Â Â Â if (rc == -1) {
> - Â Â Â Â Â Â Â Â Â Â Â syslog(LOG_ERR, "setuid: %s", strerror(errno));
> - Â Â Â Â Â Â Â Â Â Â Â goto out;
> - Â Â Â Â Â Â Â }
> - Â Â Â Â Â Â Â ccname = find_krb5_cc(CIFS_DEFAULT_KRB5_DIR, arg.creduid);
> - Â Â Â } else if (have & DKD_HAVE_UID) {
> - Â Â Â Â Â Â Â rc = setuid(arg.uid);
> - Â Â Â Â Â Â Â if (rc == -1) {
> - Â Â Â Â Â Â Â Â Â Â Â syslog(LOG_ERR, "setuid: %s", strerror(errno));
> - Â Â Â Â Â Â Â Â Â Â Â goto out;
> - Â Â Â Â Â Â Â }
> - Â Â Â Â Â Â Â ccname = find_krb5_cc(CIFS_DEFAULT_KRB5_DIR, arg.uid);
> - Â Â Â } else {
> + Â Â Â if (!legacy_uid && (have & DKD_HAVE_CREDUID))
> + Â Â Â Â Â Â Â uid = arg.creduid;
> + Â Â Â else if (have & DKD_HAVE_UID)
> + Â Â Â Â Â Â Â uid = arg.uid;
> + Â Â Â else {
> Â Â Â Â Â Â Â Â/* no uid= or creduid= parm -- something is wrong */
> Â Â Â Â Â Â Â Âsyslog(LOG_ERR, "No uid= or creduid= parm specified");
> Â Â Â Â Â Â Â Ârc = 1;
> Â Â Â Â Â Â Â Âgoto out;
> Â Â Â Â}
>
> + Â Â Â rc = setuid(uid);
> + Â Â Â if (rc == -1) {
> + Â Â Â Â Â Â Â syslog(LOG_ERR, "setuid: %s", strerror(errno));
> + Â Â Â Â Â Â Â goto out;
> + Â Â Â }
> + Â Â Â ccname = find_krb5_cc(CIFS_DEFAULT_KRB5_DIR, uid);
> +
> Â Â Â Âhost = arg.hostname;
>
> Â Â Â Â// do mech specific authorization
> --
> 1.7.3.4
>
>
ÿô.nlj·Ÿ®‰†+%ŠË±é¥Šwÿº{.nlj·¥Š{±ý¶›¡Ü}©ž²ÆzÚj:+v‰¨þø®w¥þŠàÞ¨è&¢)ß«a¶Úÿûz¹ÞúŽŠÝjÿŠwèf
