[PATCH 1/2] MIPS: Add wrappers to get/setrlimit64 to fix RLIM64_INFINITY constant
Joseph S. Myers
joseph@codesourcery.com
Wed Nov 27 01:13:00 GMT 2013
On Sat, 23 Nov 2013, Aurelien Jarno wrote:
> +# define libc_hidden_def(name) hidden_def(name)
Missing space after "hidden_def".
> +/* RLIM64_INFINITY was supposed to be a glibc convention rather than
> + * anything seen by the kernel, but it ended being passed to the kernel
> + * through the prlimit64 syscall. Given that a lot of binaries with
> + * the wrong constant value are in the wild, provide a wrapper function
> + * fixing the value after the syscall. */
No leading "*" on comment continuation lines.
> + if (__internal_getrlimit64(resource, &krlimits) < 0)
Missing space before "(".
> +libc_hidden_def(getrlimit64)
Missing space.
> +#else /* !_ABI_O32 && !_ABI_N32*/
Missing space before end of comment.
> + return __internal_setrlimit64(resource, &krlimits);
Missing space.
This patch is OK with those things fixed if it's been tested without any
local patches disabling parts of the testsuite.
--
Joseph S. Myers
joseph@codesourcery.com
More information about the Libc-alpha
mailing list