Skip to content

[v13] Defer setting up enhanced recording until after PAM has completed initializing#29578

Merged
rosstimothy merged 1 commit intobranch/v13from
bot/backport-29279-branch/v13
Jul 25, 2023
Merged

[v13] Defer setting up enhanced recording until after PAM has completed initializing#29578
rosstimothy merged 1 commit intobranch/v13from
bot/backport-29279-branch/v13

Conversation

@rosstimothy
Copy link
Copy Markdown
Contributor

Backport #29279 to branch/v13

The reexec process now has a two way wait mechanism to allow the
child process to complete any setup operations that may be required
before the parent process starts enhanced recording.

The old process was:

1) Parent launches child process
2) Child process opens PAM context and blocks on the continue signal
3) Parent sets up enhanced recording
4) Parent sends the continue signal
5) Child executes command/opens shell

The new process is:

1) Parent launches child process and waits for child continue signal
2) Child process opens PAM context and then signals it has completed setup
3) Parent receives child continue signal and sets up enhanced recording
4) Parent sends the continue signal
5) Child executes command/opens shell

Closes #29030
@rosstimothy rosstimothy marked this pull request as ready for review July 25, 2023 15:35
@github-actions github-actions Bot requested review from jakule and probakowski July 25, 2023 15:36
@rosstimothy rosstimothy added this pull request to the merge queue Jul 25, 2023
Merged via the queue into branch/v13 with commit 62941b3 Jul 25, 2023
@rosstimothy rosstimothy deleted the bot/backport-29279-branch/v13 branch July 25, 2023 16:11
@fheinecke fheinecke mentioned this pull request Sep 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants