* [PATCH] ubifs: fix RENAME_WHITEOUT support
@ 2017-03-06 9:04 Felix Fietkau
2017-03-06 9:35 ` Richard Weinberger
0 siblings, 1 reply; 4+ messages in thread
From: Felix Fietkau @ 2017-03-06 9:04 UTC (permalink / raw)
To: linux-mtd; +Cc: Richard Weinberger
Remove faulty leftover check in do_rename(), apparently introduced in a
merge that combined whiteout support changes with commit f03b8ad8d386
("fs: support RENAME_NOREPLACE for local filesystems")
Fixes: 9e0a1fff8db5 ("ubifs: Implement RENAME_WHITEOUT")
Cc: stable@vger.kernel.org
Signed-off-by: Felix Fietkau <nbd@nbd.name>
---
fs/ubifs/dir.c | 3 ---
1 file changed, 3 deletions(-)
diff --git a/fs/ubifs/dir.c b/fs/ubifs/dir.c
index 528369f3e472..f131898b7d2b 100644
--- a/fs/ubifs/dir.c
+++ b/fs/ubifs/dir.c
@@ -1316,9 +1316,6 @@ static int do_rename(struct inode *old_dir, struct dentry *old_dentry,
unsigned int uninitialized_var(saved_nlink);
struct fscrypt_name old_nm, new_nm;
- if (flags & ~RENAME_NOREPLACE)
- return -EINVAL;
-
/*
* Budget request settings: deletion direntry, new direntry, removing
* the old inode, and changing old and new parent directory inodes.
--
2.11.0
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] ubifs: fix RENAME_WHITEOUT support
2017-03-06 9:04 [PATCH] ubifs: fix RENAME_WHITEOUT support Felix Fietkau
@ 2017-03-06 9:35 ` Richard Weinberger
2017-03-06 9:46 ` Felix Fietkau
0 siblings, 1 reply; 4+ messages in thread
From: Richard Weinberger @ 2017-03-06 9:35 UTC (permalink / raw)
To: Felix Fietkau, linux-mtd
Am 06.03.2017 um 10:04 schrieb Felix Fietkau:
> Remove faulty leftover check in do_rename(), apparently introduced in a
> merge that combined whiteout support changes with commit f03b8ad8d386
> ("fs: support RENAME_NOREPLACE for local filesystems")
Good find, thank you Felix.
> Fixes: 9e0a1fff8db5 ("ubifs: Implement RENAME_WHITEOUT")
Why not f03b8ad8d386?
This commit adds the check and I oversaw that. ;-\
Thanks,
//richard
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] ubifs: fix RENAME_WHITEOUT support
2017-03-06 9:35 ` Richard Weinberger
@ 2017-03-06 9:46 ` Felix Fietkau
2017-03-29 15:26 ` Richard Weinberger
0 siblings, 1 reply; 4+ messages in thread
From: Felix Fietkau @ 2017-03-06 9:46 UTC (permalink / raw)
To: Richard Weinberger, linux-mtd
On 2017-03-06 10:35, Richard Weinberger wrote:
> Am 06.03.2017 um 10:04 schrieb Felix Fietkau:
>> Remove faulty leftover check in do_rename(), apparently introduced in a
>> merge that combined whiteout support changes with commit f03b8ad8d386
>> ("fs: support RENAME_NOREPLACE for local filesystems")
>
> Good find, thank you Felix.
>
>> Fixes: 9e0a1fff8db5 ("ubifs: Implement RENAME_WHITEOUT")
>
> Why not f03b8ad8d386?
> This commit adds the check and I oversaw that. ;-\
It seems that the merge of both commits is at fault. When I checked git
log, f03b8ad8d386 came before 9e0a1fff8db5.
- Felix
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] ubifs: fix RENAME_WHITEOUT support
2017-03-06 9:46 ` Felix Fietkau
@ 2017-03-29 15:26 ` Richard Weinberger
0 siblings, 0 replies; 4+ messages in thread
From: Richard Weinberger @ 2017-03-29 15:26 UTC (permalink / raw)
To: Felix Fietkau, linux-mtd
Felix,
Am 06.03.2017 um 10:46 schrieb Felix Fietkau:
> On 2017-03-06 10:35, Richard Weinberger wrote:
>> Am 06.03.2017 um 10:04 schrieb Felix Fietkau:
>>> Remove faulty leftover check in do_rename(), apparently introduced in a
>>> merge that combined whiteout support changes with commit f03b8ad8d386
>>> ("fs: support RENAME_NOREPLACE for local filesystems")
>>
>> Good find, thank you Felix.
>>
>>> Fixes: 9e0a1fff8db5 ("ubifs: Implement RENAME_WHITEOUT")
>>
>> Why not f03b8ad8d386?
>> This commit adds the check and I oversaw that. ;-\
> It seems that the merge of both commits is at fault. When I checked git
> log, f03b8ad8d386 came before 9e0a1fff8db5.
Yeah, the combination of both breaks it.
Let's get this into -rcX.
Thanks,
//richard
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2017-03-29 15:26 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-03-06 9:04 [PATCH] ubifs: fix RENAME_WHITEOUT support Felix Fietkau
2017-03-06 9:35 ` Richard Weinberger
2017-03-06 9:46 ` Felix Fietkau
2017-03-29 15:26 ` 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.