Re: [PATCH][1/5] Introduce proc_domode

2005-03-20 Thread Rene Scharfe
Jan Engelhardt wrote:
+#define MAXMODE		0

I think this should be 0777. SUID, SGID and sticky bit do not belong
into /proc.
It's not /proc specific, the function proc_domode can be used for all
kinds of sysctls.  It enforces a maximum mode, specified in the ->extra1
member of a sysctl table entry.  In patches 2 and 5 you can see this
value is 0444 for all five new sysctls.
Rene
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH][1/5] Introduce proc_domode

2005-03-20 Thread Jan Engelhardt

>+#define MAXMODE   0

I think this should be 0777. SUID, SGID and sticky bit do not belong into 
/proc.


Jan Engelhardt
-- 
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


[PATCH][1/5] Introduce proc_domode

2005-03-20 Thread Rene Scharfe
[The patches seem to not have made it to the list the first time,
I'm retrying without CC:s.]

Add a new sysctl proc handler , proc_domode.  It can be used to implement
sysctls for setting and/or displaying file mode or file mask values.

diff -pur l1/include/linux/sysctl.h l2/include/linux/sysctl.h
--- l1/include/linux/sysctl.h   2005-03-19 01:28:48.0 +0100
+++ l2/include/linux/sysctl.h   2005-03-19 08:08:43.0 +0100
@@ -812,6 +812,8 @@ extern int proc_doulonglongvec_minmax(ct
  void __user *, size_t *, loff_t *);
 extern int proc_doulongvec_ms_jiffies_minmax(ctl_table *table, int,
  struct file *, void __user *, size_t *, 
loff_t *);
+extern int proc_domode(ctl_table *, int, struct file *,
+   void __user *, size_t *, loff_t *);
 
 extern int do_sysctl (int __user *name, int nlen,
  void __user *oldval, size_t __user *oldlenp,
diff -pur l1/kernel/sysctl.c l2/kernel/sysctl.c
--- l1/kernel/sysctl.c  2005-03-19 01:28:49.0 +0100
+++ l2/kernel/sysctl.c  2005-03-19 08:16:41.0 +0100
@@ -2120,6 +2120,81 @@ int proc_dointvec_ms_jiffies(ctl_table *
do_proc_dointvec_ms_jiffies_conv, NULL);
 }
 
+/**
+ * proc_domode - read a file mode value
+ * @table: the sysctl table
+ * @write: %TRUE if this is a write to the sysctl file
+ * @filp: the file structure
+ * @buffer: the user buffer
+ * @lenp: the size of the user buffer
+ * @ppos: file position
+ *
+ * Reads/writes one file mode value (data type mode_t) 
+ * from/to the user buffer, treated as an ASCII string. 
+ * Optionally checks that the value fits within the mask
+ * specified with *table->extra1.  That mask cannot be
+ * bigger than 0.
+ *
+ * Returns 0 on success.
+ */
+int proc_domode(ctl_table *table, int write, struct file *filp,
+   void __user *buffer, size_t *lenp, loff_t *ppos)
+{
+#define TMPBUFLEN  6
+#define MAXMODE0
+   size_t len;
+   char __user *p;
+   char c;
+   char buf[TMPBUFLEN];
+   char *endp;
+   mode_t maxmask = MAXMODE;
+   unsigned long mode;
+   
+   if (!table->data || !*lenp || (*ppos && !write)) {
+   *lenp = 0;
+   return 0;
+   }
+   
+   if (write) {
+   if (table->extra1)
+   maxmask &= *((mode_t *)table->extra1);
+   len = 0;
+   p = buffer;
+   while (len < *lenp) {
+   if (get_user(c, p++))
+   return -EFAULT;
+   if (c == 0 || c == '\n')
+   break;
+   len++;
+   }
+   if (len > TMPBUFLEN)
+   return -EINVAL;
+   if (copy_from_user(buf, buffer, len))
+   return -EFAULT;
+   buf[len] = '\0';
+   mode = simple_strtoul(buf, &endp, 0);
+   if (mode & ~maxmask)
+   return -EPERM;
+   *((mode_t *)table->data) = mode;
+   *ppos += *lenp;
+   } else {
+   if (*((mode_t *)table->data) > MAXMODE)
+   return -EINVAL;
+   mode = *((mode_t *)table->data);
+   len = sprintf(buf, "0%03o\n", (mode_t)mode);
+   if (len > *lenp)
+   len = *lenp;
+   if (len)
+   if (copy_to_user(buffer, buf, len))
+   return -EFAULT;
+   *lenp = len;
+   *ppos += len;
+   }
+   return 0;
+#undef TMPBUFLEN
+#undef MAXMODE
+}
+
 #else /* CONFIG_PROC_FS */
 
 int proc_dostring(ctl_table *table, int write, struct file *filp,
@@ -2191,6 +2266,12 @@ int proc_doulongvec_ms_jiffies_minmax(ct
 return -ENOSYS;
 }
 
+int proc_domode(ctl_table *table, int write, struct file *filp,
+   void __user *buffer, size_t *lenp, loff_t *ppos)
+{
+   return -ENOSYS;
+}
+
 
 #endif /* CONFIG_PROC_FS */
 
@@ -2432,6 +2513,12 @@ int proc_doulongvec_ms_jiffies_minmax(ct
 return -ENOSYS;
 }
 
+int proc_domode(ctl_table *table, int write, struct file *filp,
+   void __user *buffer, size_t *lenp, loff_t *ppos)
+{
+   return -ENOSYS;
+}
+
 struct ctl_table_header * register_sysctl_table(ctl_table * table, 
int insert_at_head)
 {
@@ -2453,6 +2540,7 @@ EXPORT_SYMBOL(proc_dointvec_jiffies);
 EXPORT_SYMBOL(proc_dointvec_minmax);
 EXPORT_SYMBOL(proc_dointvec_userhz_jiffies);
 EXPORT_SYMBOL(proc_dointvec_ms_jiffies);
+EXPORT_SYMBOL(proc_domode);
 EXPORT_SYMBOL(proc_dostring);
 EXPORT_SYMBOL(proc_doulongvec_minmax);
 EXPORT_SYMBOL(proc_doulonglongvec_minmax);

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.ker