Skip to content

fix(pcrlock): use .d path for PCRLOCK_KERNEL_*_PATH - #415

Open
deepin-ci-robot wants to merge 1 commit into
deepin-community:masterfrom
deepin-ci-robot:backport/fix-pcrlock-use-d-path-for-kernel-pcrlock
Open

fix(pcrlock): use .d path for PCRLOCK_KERNEL_*_PATH#415
deepin-ci-robot wants to merge 1 commit into
deepin-community:masterfrom
deepin-ci-robot:backport/fix-pcrlock-use-d-path-for-kernel-pcrlock

Conversation

@deepin-ci-robot

@deepin-ci-robot deepin-ci-robot commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

Problem Before the Change

systemd-pcrlock used incorrect paths for the generated kernel cmdline / initrd component files: PCRLOCK_KERNEL_CMDLINE_PATH and PCRLOCK_KERNEL_INITRD_PATH pointed to .../710-kernel-cmdline.pcrlock/generated.pcrlock and .../720-kernel-initrd.pcrlock/generated.pcrlock instead of the .pcrlock.d directories used by all other generated components. The tool then fails with:

Failed to parse component file /var/lib/pcrlock.d/720-kernel-initrd.pcrlock, ignoring: Is a directory

What This PR Changes

  • Backport upstream fix that corrects PCRLOCK_KERNEL_CMDLINE_PATH and PCRLOCK_KERNEL_INITRD_PATH in src/pcrlock/pcrlock.c to use the .pcrlock.d path suffix, consistent with the other component paths.

Problem Solved After the Change

systemd-pcrlock no longer treats /var/lib/pcrlock.d/720-kernel-initrd.pcrlock as a stray directory and correctly reads/writes the generated kernel cmdline and initrd pcrlock component files.

Changes

  • Add debian/patches/fix-pcrlock-use-d-path-for-kernel-pcrlock.patch
  • Modify debian/patches/series
  • Modify debian/changelog

Upstream

Fix the path for the generated.pcrlock files for the cmdline and initrd
cases. Without it the tool complains with:

    Failed to parse component file /var/lib/pcrlock.d/720-kernel-initrd.pcrlock, ignoring: Is a directory

Changes:
  - Add debian/patches/fix-pcrlock-use-d-path-for-kernel-pcrlock.patch
  - Modify debian/patches/series
  - Modify debian/changelog

Upstream: systemd/systemd@ef94944

Generated-By: glm-5.3-flash
Co-Authored-By: deepin-ci-robot <packages@deepin.org>
@deepin-ci-robot

Copy link
Copy Markdown
Contributor Author

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign zccrs for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@github-actions

github-actions Bot commented Sep 3, 2026

Copy link
Copy Markdown

TAG Bot

TAG: 255.2-4deepin68
EXISTED: no
DISTRIBUTION: unstable

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.

1 participant