Skip to content

Add online resize support for ocfs2, take 4

Mark Fasheh ocfs2-devel 1 version New

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest0 of 4 patches 0 repliesNew

Patches in v1 No applies recorded yet

None of the 4 patches the cover letter announces are in this archive.

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.

Read the v1 thread

Cover letter [Ocfs2-devel] [PATCH 0/4] Add online resize support for ocfs2, take 4
Quoted 2 lines · tao.ma

Mark,
Wish this patch series meets your need.

Ok, kernel patches are committed to ocfs2.git. I added this patch to the end of the series, please check.

	--Mark

--
Mark Fasheh
Principal Software Developer, Oracle
[email protected]

From: Mark Fasheh <[email protected]>

ocfs2: Add missing permission checks

Check that an online resize is being driven by a user with permission to change system resource limits.

Signed-off-by Mark Fasheh <[email protected]>
---
 fs/ocfs2/ioctl.c |    6 ++++++
 1 files changed, 6 insertions(+), 0 deletions(-)
fs/ocfs2/ioctl.c +6 -0
index a4dda60..83b3e0f 100644
@@ -144,12 +144,18 @@ int ocfs2_ioctl(struct inode * inode, struct file * filp,
 
 		return ocfs2_change_file_space(filp, cmd, &sr);
 	case OCFS2_IOC_GROUP_EXTEND:
+		if (!capable(CAP_SYS_RESOURCE))
+			return -EPERM;
+
 		if (get_user(new_clusters, (int __user *)arg))
 			return -EFAULT;
 
 		return ocfs2_group_extend(inode, new_clusters);
 	case OCFS2_IOC_GROUP_ADD:
 	case OCFS2_IOC_GROUP_ADD64:
+		if (!capable(CAP_SYS_RESOURCE))
+			return -EPERM;
+
 		if (copy_from_user(&input, (int __user *) arg, sizeof(input)))
 			return -EFAULT;
Signature · 1 line
1.5.3.6

Read the cover letter in its thread