[LTP] [PATCH v2] setpgid02: use 1 instead of getpgid(1)

[Edward Liaw]

On Fri, Apr 7, 2023 at 3:18 AM Teo Couprie Diaz <teo.coupriediaz@arm.com> wrote:
> However, I have encountered an issue on the same check of this test,
> unrelated to Edward's issue.
>
> Indeed, on systems that run the shell as PID 1 (for example a basic
> busybox rootfs) the EPERM check wouldn't work.
> This is because LTP would run within the same session ID as init, which
> would allow the test to change the PGID and not trigger the EPERM.
>
> I am working on a patch and wanted to get some input. My current idea
> would be to fork a child that would create a new session and try to
> setpgid() the child.
> This would also allow to use the main process' PGID, as it would be in
> another session from the child anyway. This would remove the need to
> getpgid() init, which hopefully should fix your issue on Android as well.
>

That makes sense to me, but it seems to me that setpgid03 is already
testing it that way.

> However, this adds a lot more complexity in the test: needing to fork
> and synchronize with the child as the main process needs to wait for the
> child to change its session ID, otherwise the test would fail.
>
> Do you think this idea makes sense ? I would send it for review once I
> ironed out the patch.
> Another solution would be for LTP to change its session ID by default,
> which would prevent the need for a change to setpgid02 on top of Edward's.
> However, I don't fully understand the possible consequences of having
> LTP change its SID for all tests.
>

Alternatively, maybe it could be reverted to using the hardcoded 99999
as an invalid PGID as it was before Avinesh's patch or the test case
removed because it is handled in setpgid03?

Thanks,
Edward