X-Git-Url: https://git.whamcloud.com/?a=blobdiff_plain;f=lustre%2Fofd%2Fofd_objects.c;h=9f760813c8e85fe025bea01fea6a085863a46e2f;hb=45900a7777ac02130d8bf65724c4b6cffca9d546;hp=f838e2317993ad0fd30f5eabfd93a742154956a4;hpb=03b86ce064c224ed6ed77675183bfd35d493f918;p=fs%2Flustre-release.git

diff --git a/lustre/ofd/ofd_objects.c b/lustre/ofd/ofd_objects.c
index f838e23..9f76081 100644
--- a/lustre/ofd/ofd_objects.c
+++ b/lustre/ofd/ofd_objects.c
@@ -41,7 +41,6 @@
 #define DEBUG_SUBSYSTEM S_FILTER
 
 #include <dt_object.h>
-#include <lustre/lustre_idl.h>
 #include <lustre_lfsck.h>
 
 #include "ofd_internal.h"
@@ -225,7 +224,7 @@ int ofd_precreate_objects(const struct lu_env *env, struct ofd_device *ofd,
 		RETURN(-ENOMEM);
 
 	info->fti_attr.la_valid = LA_TYPE | LA_MODE;
-	info->fti_attr.la_mode = S_IFREG | S_ISUID | S_ISGID | 0666;
+	info->fti_attr.la_mode = S_IFREG | S_ISUID | S_ISGID | S_ISVTX | 0666;
 	info->fti_dof.dof_type = dt_mode_to_dft(S_IFREG);
 
 	info->fti_attr.la_valid |= LA_ATIME | LA_MTIME | LA_CTIME;
@@ -410,9 +409,9 @@ out:
  *
  * If the object still has SUID+SGID bits set, meaning that it was precreated
  * by the MDT before it was assigned to any file, (see ofd_precreate_objects())
- * then we will accept the UID+GID if sent by the client for initializing the
- * ownership of this object.  We only allow this to happen once (so clear these
- * bits) and later only allow setattr.
+ * then we will accept the UID/GID/PROJID if sent by the client for initializing
+ * the ownership of this object.  We only allow this to happen once (so clear
+ * these bits) and later only allow setattr.
  *
  * \param[in] env	 execution environment
  * \param[in] fo	 OFD object
@@ -422,7 +421,7 @@ out:
  * \retval		0 if successful
  * \retval		negative value on error
  */
-int ofd_attr_handle_ugid(const struct lu_env *env, struct ofd_object *fo,
+int ofd_attr_handle_id(const struct lu_env *env, struct ofd_object *fo,
 			 struct lu_attr *la, int is_setattr)
 {
 	struct ofd_thread_info	*info = ofd_info(env);
@@ -432,7 +431,8 @@ int ofd_attr_handle_ugid(const struct lu_env *env, struct ofd_object *fo,
 
 	ENTRY;
 
-	if (!(la->la_valid & LA_UID) && !(la->la_valid & LA_GID))
+	if (!(la->la_valid & LA_UID) && !(la->la_valid & LA_GID) &&
+	    !(la->la_valid & LA_PROJID))
 		RETURN(0);
 
 	rc = dt_attr_get(env, ofd_object_child(fo), ln);
@@ -441,17 +441,27 @@ int ofd_attr_handle_ugid(const struct lu_env *env, struct ofd_object *fo,
 
 	LASSERT(ln->la_valid & LA_MODE);
 
+	/*
+	 * Only allow setattr to change UID/GID/PROJID, if
+	 * SUID+SGID is not set which means this is not
+	 * initialization of this objects.
+	 */
 	if (!is_setattr) {
 		if (!(ln->la_mode & S_ISUID))
 			la->la_valid &= ~LA_UID;
 		if (!(ln->la_mode & S_ISGID))
 			la->la_valid &= ~LA_GID;
+		if (!(ln->la_mode & S_ISVTX))
+			la->la_valid &= ~LA_PROJID;
 	}
 
+	/* Initialize ownership of this object, clear SUID+SGID bits*/
 	if ((la->la_valid & LA_UID) && (ln->la_mode & S_ISUID))
 		mask |= S_ISUID;
 	if ((la->la_valid & LA_GID) && (ln->la_mode & S_ISGID))
 		mask |= S_ISGID;
+	if ((la->la_valid & LA_PROJID) && (ln->la_mode & S_ISVTX))
+		mask |= S_ISVTX;
 	if (mask != 0) {
 		if (!(la->la_valid & LA_MODE) || !is_setattr) {
 			la->la_mode = ln->la_mode;
@@ -507,7 +517,7 @@ int ofd_attr_set(const struct lu_env *env, struct ofd_object *fo,
 	if (rc)
 		GOTO(unlock, rc);
 
-	rc = ofd_attr_handle_ugid(env, fo, la, 1 /* is_setattr */);
+	rc = ofd_attr_handle_id(env, fo, la, 1 /* is_setattr */);
 	if (rc != 0)
 		GOTO(unlock, rc);
 
@@ -633,7 +643,7 @@ int ofd_object_punch(const struct lu_env *env, struct ofd_object *fo,
 	if (rc)
 		GOTO(unlock, rc);
 
-	rc = ofd_attr_handle_ugid(env, fo, la, 0 /* !is_setattr */);
+	rc = ofd_attr_handle_id(env, fo, la, 0 /* !is_setattr */);
 	if (rc != 0)
 		GOTO(unlock, rc);
 
@@ -723,7 +733,7 @@ unlock:
  * \retval		0 if successful
  * \retval		negative value on error
  */
-int ofd_object_destroy(const struct lu_env *env, struct ofd_object *fo,
+int ofd_destroy(const struct lu_env *env, struct ofd_object *fo,
 		       int orphan)
 {
 	struct ofd_device	*ofd = ofd_obj2dev(fo);