Wouldn't it give more flexibility to the upcall program if we sent both or changed the name (and be less confusing) e.g. to "mount_uid" or something else distinct<br><br><div class="gmail_quote">On Mon, Aug 3, 2009 at 1:34 PM, Jeff Layton <span dir="ltr"><<a href="mailto:jlayton@redhat.com">jlayton@redhat.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">The ownership of files on the mount has no direct relationship to the<br>
credentials used to do the mount. Instead of sending the uid<br>
corresponding to the owner of files on the mount, instead send the real<br>
uid of the process that initiated the upcall.<br>
<br>
Usually this will be the real uid of the process running /bin/mount.<br>
Eventually however, we want to be able to establish new sessions for<br>
users that walk into a cifs mount. For that we need the real uid of<br>
those users.<br>
<br>
Signed-off-by: Jeff Layton <<a href="mailto:jlayton@redhat.com">jlayton@redhat.com</a>><br>
---<br>
fs/cifs/cifs_spnego.c | 2 +-<br>
1 files changed, 1 insertions(+), 1 deletions(-)<br>
<br>
diff --git a/fs/cifs/cifs_spnego.c b/fs/cifs/cifs_spnego.c<br>
index 8ec7736..1d0d8fc 100644<br>
--- a/fs/cifs/cifs_spnego.c<br>
+++ b/fs/cifs/cifs_spnego.c<br>
@@ -140,7 +140,7 @@ cifs_get_spnego_key(struct cifsSesInfo *sesInfo)<br>
goto out;<br>
<br>
dp = description + strlen(description);<br>
- sprintf(dp, ";uid=0x%x", sesInfo->linux_uid);<br>
+ sprintf(dp, ";uid=0x%x", current_uid());<br>
<br>
dp = description + strlen(description);<br>
sprintf(dp, ";user=%s", sesInfo->userName);<br>
<font color="#888888">--<br>
1.6.0.6<br>
<br>
</font></blockquote></div><br><br clear="all"><br>-- <br>Thanks,<br><br>Steve<br>