Re: [PATCH v3 0/4] MR6855: server: Fix renaming file to a hardlink of itself.

[Jinoh Kang (＠iamahuman)]

On Mon Nov 25 19:19:31 2024 +0000, Gabriel Ivăncescu wrote:
> I haven't changed this, mostly because I don't know exactly what you
> meant. I updated with changes and the cached idea, let me know what you
> think when you're back.

Like, wine (regardless of this patch) is doing `free( name );` and then early return. That doesn't *look* right to me. I feel like we're still supposed to do the following checks (e.g., permissions check, get_inode check, etc) even if src and dest are the same file. But the current tests are too lacking to verify if what I'm claiming is true.