Had a cursory look and the series looks good to me, it make sense.
Small nit : patch 3/3, "&if" in the commit message instead of "&ifr"
On 26/10/2023 13:23, Kevin Brodsky wrote:
Hi,
This series fixes a number of issues that the switch to checked (capability-based) uaccess [1] has revealed. These patches complete those that I have posted upstream [2] (and are already merged); they address issues that are likely to be too purecap-specific for upstream.
Review branch:
https://git.morello-project.org/kbrodsky-arm/morello-linux-ltp/-/commits/rev...
Cheers, Kevin
[1] https://op-lists.linaro.org/archives/list/linux-morello@op-lists.linaro.org/... [2] https://lore.kernel.org/ltp/20231023135647.2157030-1-kevin.brodsky@arm.com/
Kevin Brodsky (3): syscalls/clone3: Avoid truncating pointers in purecap syscalls/prctl: Fix invalid pointer cast in purecap syscalls/sockioctl: Avoid out-of-bound uaccess in purecap
testcases/kernel/syscalls/clone3/clone301.c | 6 +++--- testcases/kernel/syscalls/prctl/prctl02.c | 15 +++++++++------ testcases/kernel/syscalls/sockioctl/sockioctl01.c | 2 +- 3 files changed, 13 insertions(+), 10 deletions(-)