Skip to content

[stable34] fix(encryption): use the block size of the file encoding when fixing sizes - #65459

Open
backportbot[bot] wants to merge 1 commit into
stable34from
backport/65426/stable34
Open

backportbot[bot] wants to merge 1 commit into
stable34from
backport/65426/stable34

Conversation

@backportbot

@backportbot backportbot Bot commented Oct 13, 2026 •

Copy link
Copy Markdown

Backport of #65426

Warning, This backport's changes differ from the original and might be incomplete ⚠️

Todo

  • Review and resolve any conflicts
  • Review and verify the backported changes
  • Amend HEAD commit to remove the line stating to skip CI

Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

…sizes

`fixUnencryptedSize()` asked the encryption module for the unencrypted
block size before calling `begin()`. The module only learns the encoding
of the file (binary or legacy base64) from the header in `begin()`, so
the block size of binary encoded files was used for legacy base64 files.

This stored an inflated unencrypted size for legacy files, which then
breaks downloads: the announced content length is too large and the
signature check of the last block fails with "Bad Signature".

Fixes #63662

Assisted-by: ClaudeCode:claude-opus-5-5
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
@CarlSchwan
CarlSchwan force-pushed the backport/65426/stable34 branch from 422526b to 6eeb2a7 Compare October 14, 2026 07:55
@CarlSchwan
CarlSchwan marked this pull request as ready for review October 14, 2026 07:55
@CarlSchwan
CarlSchwan requested a review from a team as a code owner October 14, 2026 07:55
@CarlSchwan
CarlSchwan requested review from Altahrim, icewind1991 and sorbaugh and removed request for a team October 14, 2026 07:55

This branch has not been deployed

No deployments
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.

2 participants