|
Message-ID: <Zjlq0nlT6POMfWb8@kodidev-ubuntu> Date: Mon, 6 May 2024 16:42:10 -0700 From: Tony Ambardar <tony.ambardar@...il.com> To: Rich Felker <dalias@...c.org> Cc: musl@...ts.openwall.com Subject: Re: [PATCH v2] add renameat2 linux syscall wrapper On Mon, May 06, 2024 at 10:50:57AM -0400, Rich Felker wrote: > On Tue, Apr 23, 2024 at 04:43:55PM -0700, Tony Ambardar wrote: > > This syscall is available since Linux 3.15 and also implemented in glibc > > from version 2.28. It is commonly used in filesystem or security contexts. > > [SNIP] > > If flags is 0, the SYS_renameat syscall is semantically equivalent to > the SYS_renameat2 one, so it would be better to just unconditionally > do that first rather than failing and falling back. > Do you mean rearranging and dropping the ENOSYS conditional, e.g. something like below? > int r; > #ifdef SYS_renameat > if (!flags) r = __syscall(SYS_renameat, oldfd, old, newfd, new); > else > #endif > r = __syscall(SYS_renameat2, oldfd, old, newfd, new, flags); > return __syscall_ret(r); Please clarify and I'll update. Thanks, Tony > Rich
Powered by blists - more mailing lists
Confused about mailing lists and their use? Read about mailing lists on Wikipedia and check out these guidelines on proper formatting of your messages.