Generally LGTM, I also added a focal task to the bug to later SRU it once groovy is fixed.
Changelog, patch headers and such look good.
The only thing that I found is that there was a follow on fix which IMHO should be included as well: => https://gitlab.com/apparmor/apparmor/-/commit/ef591a67cedc1da0676b26448ea96fa8c073c253 Due to that the state is "needs fixing", but other than this is already seems to be good.
Finally I'll add a review-slot for Ubuntu-security. That isn't strictly needed, but usually a good practice on security related changes.
« Back to merge proposal
Generally LGTM, I also added a focal task to the bug to later SRU it once groovy is fixed.
Changelog, patch headers and such look good.
The only thing that I found is that there was a follow on fix which IMHO should be included as well: /gitlab. com/apparmor/ apparmor/ -/commit/ ef591a67cedc1da 0676b26448ea96f a8c073c253
=> https:/
Due to that the state is "needs fixing", but other than this is already seems to be good.
Finally I'll add a review-slot for Ubuntu-security. That isn't strictly needed, but usually a good practice on security related changes.