Skip to content

fix: Clear setuid/setgid bits when downloading files via scp (CVE-2026-35385)#16

Open
deepin-ci-robot wants to merge 2 commits into
masterfrom
fix/CVE-2026-35385
Open

fix: Clear setuid/setgid bits when downloading files via scp (CVE-2026-35385)#16
deepin-ci-robot wants to merge 2 commits into
masterfrom
fix/CVE-2026-35385

Conversation

@deepin-ci-robot
Copy link
Copy Markdown
Contributor

CVE-2026-35385 Fix

Vulnerability

When downloading files as root in legacy (-O) mode and without the -p (preserve modes) flag set, scp did not clear setuid/setgid bits from downloaded files as one might typically expect. This bug dates back to the original Berkeley rcp program.

Fix

Backport upstream commit 487e8ac which adds mask |= 07000 to clear setuid/setgid/sticky bits when not preserving modes.

Changes

  • debian/patches/0038-upstream-Clear-setuid-setgid-bits-when-downloading-.patch: New patch
  • debian/patches/series: Add new patch
  • debian/changelog: Bump to 1:9.9p2-0deepin7

References

…6-35385)

When downloading files as root in legacy (-O) mode and without the -p
(preserve modes) flag set, scp did not clear setuid/setgid bits from
downloaded files as one might typically expect.

This bug dates back to the original Berkeley rcp program.

Reported by Christos Papakonstantinou of Cantina and Spearbit.

Upstream commit: 487e8ac146f7d6616f65c125d5edb210519b833a

Co-Authored-By: hudeng <hudeng@deepin.org>
@github-actions
Copy link
Copy Markdown

TAG Bot

TAG: 1%9.9p2-0deepin7
EXISTED: no
DISTRIBUTION: unstable

@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 yukarichiba 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

@deepin-ci-robot
Copy link
Copy Markdown
Contributor Author

/hold
因为该quilt包的上游版本号变更,详情见: deepin-community/infra-settings#134

@Zeno-sole
Copy link
Copy Markdown

/integrate

@github-actions
Copy link
Copy Markdown

AutoIntegrationPr Bot
auto integrate with pr url: deepin-community/Repository-Integration#4062
PrNumber: 4062
PrBranch: auto-integration-26216164277

The patch had offset 7 lines and fuzz 1 when applied. Refreshed with
quilt to align hunk context with current codebase.

Upstream: openssh/openssh-portable@487e8ac
Generated-By: uos/glm-5.1
Co-Authored-By: hudeng <hudeng@deepin.org>
@hudeng-go
Copy link
Copy Markdown
Contributor

/integrate

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