Skip to content

Conversation

@larsewi
Copy link
Contributor

@larsewi larsewi commented Dec 2, 2025

Ticket: ENT-13535
Signed-off-by: Lars Erik Wik lars.erik.wik@northern.tech
(cherry picked from commit e865642)

Backported from #5967

@larsewi larsewi marked this pull request as ready for review December 2, 2025 09:16
@larsewi
Copy link
Contributor Author

larsewi commented Dec 2, 2025

@cf-bottom Jenkins with exotics please :)

@cf-bottom
Copy link

Signed-off-by: Lars Erik Wik <lars.erik.wik@northern.tech>
(cherry picked from commit ef1f52a)
I also refactored the code to use a dynamic buffer when assembling the
commands. Estimating the buffer size needed is error prone and can
quickly lead to buffer overflows.

Ticket: ENT-13535
Changelog: Title
Signed-off-by: Lars Erik Wik <lars.erik.wik@northern.tech>
(cherry picked from commit aaf314a)
@larsewi
Copy link
Contributor Author

larsewi commented Dec 4, 2025

@cf-bottom Jenkins please :)

@cf-bottom
Copy link

… quoting arguments

Ticket: ENT-13535
Changelog: none
(cherry picked from commit 6f97d0a)
Signed-off-by: Lars Erik Wik <lars.erik.wik@northern.tech>
@larsewi
Copy link
Contributor Author

larsewi commented Dec 5, 2025

@cf-bottom Jenkins please :)

@cf-bottom
Copy link

@larsewi larsewi merged commit 18f89e2 into cfengine:3.21.x Dec 7, 2025
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants