* [PATCH] ubifs: setflags: Don't show error message when vfs_ioc_setflags_prepare() fails
@ 2020-08-28 3:32 ` Zhihao Cheng
0 siblings, 0 replies; 6+ messages in thread
From: Zhihao Cheng @ 2020-08-28 3:32 UTC (permalink / raw)
To: richard.weinberger, yi.zhang; +Cc: linux-kernel, linux-mtd
Following process will trigger ubifs_err:
1. useradd -m freg (Under root)
2. cd /home/freg && mkdir mp (Under freg)
3. mount -t ubifs /dev/ubi0_0 /home/freg/mp (Under root)
4. cd /home/freg && echo 123 > mp/a (Under root)
5. cd mp && chown freg a && chgrp freg a && chmod 777 a (Under root)
6. chattr +i a (Under freg)
UBIFS error (ubi0:0 pid 1723): ubifs_ioctl [ubifs]: can't modify inode
65 attributes
chattr: Operation not permitted while setting flags on a
This is not an UBIFS problem, it was caused by task priviliage checking
on file operations. Remove error message printing from kernel just like
other filesystems (eg. ext4), since we already have enough information
from userspace tools.
Signed-off-by: Zhihao Cheng <chengzhihao1@huawei.com>
---
fs/ubifs/ioctl.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/fs/ubifs/ioctl.c b/fs/ubifs/ioctl.c
index 3df9be2c684c..4363d85a3fd4 100644
--- a/fs/ubifs/ioctl.c
+++ b/fs/ubifs/ioctl.c
@@ -134,7 +134,6 @@ static int setflags(struct inode *inode, int flags)
return err;
out_unlock:
- ubifs_err(c, "can't modify inode %lu attributes", inode->i_ino);
mutex_unlock(&ui->ui_mutex);
ubifs_release_budget(c, &req);
return err;
--
2.25.4
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH] ubifs: setflags: Don't show error message when vfs_ioc_setflags_prepare() fails
@ 2020-08-28 3:32 ` Zhihao Cheng
0 siblings, 0 replies; 6+ messages in thread
From: Zhihao Cheng @ 2020-08-28 3:32 UTC (permalink / raw)
To: richard.weinberger, yi.zhang; +Cc: linux-mtd, linux-kernel
Following process will trigger ubifs_err:
1. useradd -m freg (Under root)
2. cd /home/freg && mkdir mp (Under freg)
3. mount -t ubifs /dev/ubi0_0 /home/freg/mp (Under root)
4. cd /home/freg && echo 123 > mp/a (Under root)
5. cd mp && chown freg a && chgrp freg a && chmod 777 a (Under root)
6. chattr +i a (Under freg)
UBIFS error (ubi0:0 pid 1723): ubifs_ioctl [ubifs]: can't modify inode
65 attributes
chattr: Operation not permitted while setting flags on a
This is not an UBIFS problem, it was caused by task priviliage checking
on file operations. Remove error message printing from kernel just like
other filesystems (eg. ext4), since we already have enough information
from userspace tools.
Signed-off-by: Zhihao Cheng <chengzhihao1@huawei.com>
---
fs/ubifs/ioctl.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/fs/ubifs/ioctl.c b/fs/ubifs/ioctl.c
index 3df9be2c684c..4363d85a3fd4 100644
--- a/fs/ubifs/ioctl.c
+++ b/fs/ubifs/ioctl.c
@@ -134,7 +134,6 @@ static int setflags(struct inode *inode, int flags)
return err;
out_unlock:
- ubifs_err(c, "can't modify inode %lu attributes", inode->i_ino);
mutex_unlock(&ui->ui_mutex);
ubifs_release_budget(c, &req);
return err;
--
2.25.4
______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] ubifs: setflags: Don't show error message when vfs_ioc_setflags_prepare() fails
2020-08-28 3:32 ` Zhihao Cheng
@ 2020-08-30 19:23 ` Richard Weinberger
-1 siblings, 0 replies; 6+ messages in thread
From: Richard Weinberger @ 2020-08-30 19:23 UTC (permalink / raw)
To: Zhihao Cheng; +Cc: zhangyi (F), LKML, linux-mtd
On Fri, Aug 28, 2020 at 5:33 AM Zhihao Cheng <chengzhihao1@huawei.com> wrote:
>
> Following process will trigger ubifs_err:
> 1. useradd -m freg (Under root)
> 2. cd /home/freg && mkdir mp (Under freg)
> 3. mount -t ubifs /dev/ubi0_0 /home/freg/mp (Under root)
> 4. cd /home/freg && echo 123 > mp/a (Under root)
> 5. cd mp && chown freg a && chgrp freg a && chmod 777 a (Under root)
> 6. chattr +i a (Under freg)
>
> UBIFS error (ubi0:0 pid 1723): ubifs_ioctl [ubifs]: can't modify inode
> 65 attributes
> chattr: Operation not permitted while setting flags on a
>
> This is not an UBIFS problem, it was caused by task priviliage checking
> on file operations. Remove error message printing from kernel just like
> other filesystems (eg. ext4), since we already have enough information
> from userspace tools.
>
> Signed-off-by: Zhihao Cheng <chengzhihao1@huawei.com>
> ---
> fs/ubifs/ioctl.c | 1 -
> 1 file changed, 1 deletion(-)
Makes sense, thanks for fixing!
BTW: I didn't forget about the other valuable fixes from you, in the
next two weeks I'll hopefully
find more time for reviewing/testing. :-)
--
Thanks,
//richard
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] ubifs: setflags: Don't show error message when vfs_ioc_setflags_prepare() fails
@ 2020-08-30 19:23 ` Richard Weinberger
0 siblings, 0 replies; 6+ messages in thread
From: Richard Weinberger @ 2020-08-30 19:23 UTC (permalink / raw)
To: Zhihao Cheng; +Cc: linux-mtd, LKML, zhangyi (F)
On Fri, Aug 28, 2020 at 5:33 AM Zhihao Cheng <chengzhihao1@huawei.com> wrote:
>
> Following process will trigger ubifs_err:
> 1. useradd -m freg (Under root)
> 2. cd /home/freg && mkdir mp (Under freg)
> 3. mount -t ubifs /dev/ubi0_0 /home/freg/mp (Under root)
> 4. cd /home/freg && echo 123 > mp/a (Under root)
> 5. cd mp && chown freg a && chgrp freg a && chmod 777 a (Under root)
> 6. chattr +i a (Under freg)
>
> UBIFS error (ubi0:0 pid 1723): ubifs_ioctl [ubifs]: can't modify inode
> 65 attributes
> chattr: Operation not permitted while setting flags on a
>
> This is not an UBIFS problem, it was caused by task priviliage checking
> on file operations. Remove error message printing from kernel just like
> other filesystems (eg. ext4), since we already have enough information
> from userspace tools.
>
> Signed-off-by: Zhihao Cheng <chengzhihao1@huawei.com>
> ---
> fs/ubifs/ioctl.c | 1 -
> 1 file changed, 1 deletion(-)
Makes sense, thanks for fixing!
BTW: I didn't forget about the other valuable fixes from you, in the
next two weeks I'll hopefully
find more time for reviewing/testing. :-)
--
Thanks,
//richard
______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] ubifs: setflags: Don't show error message when vfs_ioc_setflags_prepare() fails
2020-08-30 19:23 ` Richard Weinberger
@ 2020-09-13 18:39 ` Richard Weinberger
-1 siblings, 0 replies; 6+ messages in thread
From: Richard Weinberger @ 2020-09-13 18:39 UTC (permalink / raw)
To: Zhihao Cheng; +Cc: zhangyi (F), LKML, linux-mtd
On Sun, Aug 30, 2020 at 9:23 PM Richard Weinberger
<richard.weinberger@gmail.com> wrote:
>
> On Fri, Aug 28, 2020 at 5:33 AM Zhihao Cheng <chengzhihao1@huawei.com> wrote:
> >
> > Following process will trigger ubifs_err:
> > 1. useradd -m freg (Under root)
> > 2. cd /home/freg && mkdir mp (Under freg)
> > 3. mount -t ubifs /dev/ubi0_0 /home/freg/mp (Under root)
> > 4. cd /home/freg && echo 123 > mp/a (Under root)
> > 5. cd mp && chown freg a && chgrp freg a && chmod 777 a (Under root)
> > 6. chattr +i a (Under freg)
> >
> > UBIFS error (ubi0:0 pid 1723): ubifs_ioctl [ubifs]: can't modify inode
> > 65 attributes
> > chattr: Operation not permitted while setting flags on a
> >
> > This is not an UBIFS problem, it was caused by task priviliage checking
> > on file operations. Remove error message printing from kernel just like
> > other filesystems (eg. ext4), since we already have enough information
> > from userspace tools.
> >
> > Signed-off-by: Zhihao Cheng <chengzhihao1@huawei.com>
> > ---
> > fs/ubifs/ioctl.c | 1 -
> > 1 file changed, 1 deletion(-)
>
> Makes sense, thanks for fixing!
Applied to fixes.
--
Thanks,
//richard
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] ubifs: setflags: Don't show error message when vfs_ioc_setflags_prepare() fails
@ 2020-09-13 18:39 ` Richard Weinberger
0 siblings, 0 replies; 6+ messages in thread
From: Richard Weinberger @ 2020-09-13 18:39 UTC (permalink / raw)
To: Zhihao Cheng; +Cc: linux-mtd, LKML, zhangyi (F)
On Sun, Aug 30, 2020 at 9:23 PM Richard Weinberger
<richard.weinberger@gmail.com> wrote:
>
> On Fri, Aug 28, 2020 at 5:33 AM Zhihao Cheng <chengzhihao1@huawei.com> wrote:
> >
> > Following process will trigger ubifs_err:
> > 1. useradd -m freg (Under root)
> > 2. cd /home/freg && mkdir mp (Under freg)
> > 3. mount -t ubifs /dev/ubi0_0 /home/freg/mp (Under root)
> > 4. cd /home/freg && echo 123 > mp/a (Under root)
> > 5. cd mp && chown freg a && chgrp freg a && chmod 777 a (Under root)
> > 6. chattr +i a (Under freg)
> >
> > UBIFS error (ubi0:0 pid 1723): ubifs_ioctl [ubifs]: can't modify inode
> > 65 attributes
> > chattr: Operation not permitted while setting flags on a
> >
> > This is not an UBIFS problem, it was caused by task priviliage checking
> > on file operations. Remove error message printing from kernel just like
> > other filesystems (eg. ext4), since we already have enough information
> > from userspace tools.
> >
> > Signed-off-by: Zhihao Cheng <chengzhihao1@huawei.com>
> > ---
> > fs/ubifs/ioctl.c | 1 -
> > 1 file changed, 1 deletion(-)
>
> Makes sense, thanks for fixing!
Applied to fixes.
--
Thanks,
//richard
______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2020-09-13 18:40 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-08-28 3:32 [PATCH] ubifs: setflags: Don't show error message when vfs_ioc_setflags_prepare() fails Zhihao Cheng
2020-08-28 3:32 ` Zhihao Cheng
2020-08-30 19:23 ` Richard Weinberger
2020-08-30 19:23 ` Richard Weinberger
2020-09-13 18:39 ` Richard Weinberger
2020-09-13 18:39 ` Richard Weinberger
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.