On Mon, Aug 17, 2020 at 05:04:23PM -0500, Eric W. Biederman wrote: > Simplify the code, and remove the chance of races by reading > RLIMIT_NOFILE only once in f_dupfd. > > Pass the read value of RLIMIT_NOFILE into alloc_fd which is the other > location the rlimit was read in f_dupfd. As f_dupfd is the only > caller of alloc_fd this changing alloc_fd is trivially safe. > > Further this causes alloc_fd to take all of the same arguments as > __alloc_fd except for the files_struct argument. > > Signed-off-by: "Eric W. Biederman" <ebiederm@xxxxxxxxxxxx> > --- Acked-by: Christian Brauner <christian.brauner@xxxxxxxxxx>