Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

try different permutation to try to recover the broken checksum #5572

Merged
merged 1 commit into from
Apr 4, 2023

Conversation

mgallien
Copy link
Collaborator

@mgallien mgallien commented Apr 4, 2023

will only accept a checksum that can be computed from proper data

should still be safe

@mgallien
Copy link
Collaborator Author

mgallien commented Apr 4, 2023

/backport to stable-3.8

@claucambra
Copy link
Collaborator

Some comments would be nice to explain the bug and the ensuing fix

@mgallien mgallien added this to the 3.9.0 milestone Apr 4, 2023
src/libsync/clientsideencryption.cpp Outdated Show resolved Hide resolved
src/libsync/clientsideencryption.cpp Outdated Show resolved Hide resolved
@mgallien mgallien force-pushed the bugfix/tryHardToRecoverBrokenE2eeChecksums branch from 7c813c3 to b8679cf Compare April 4, 2023 15:40
will only accept a checksum that can be computed from proper data

should still be safe

Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>
@mgallien mgallien force-pushed the bugfix/tryHardToRecoverBrokenE2eeChecksums branch from b8679cf to b0c6948 Compare April 4, 2023 15:41
@mgallien mgallien requested a review from camilasan April 4, 2023 15:43
@mgallien mgallien merged commit 9e7197a into master Apr 4, 2023
@mgallien mgallien deleted the bugfix/tryHardToRecoverBrokenE2eeChecksums branch April 4, 2023 15:47
@nextcloud-desktop-bot
Copy link

AppImage file: nextcloud-PR-5572-b0c69481f2f457d88389b70ad511ba356f03b9bb-x86_64.AppImage

To test this change/fix you can simply download above AppImage file and test it.

Please make sure to quit your existing Nextcloud app and backup your data.

@timebuzzerfelix
Copy link

timebuzzerfelix commented Apr 13, 2023

I tried this, first looked good, but then it downloaded some random files (that were already local) and after that marked some of this files with the same error #5564
I had 1 file with this error before trying this fix, after I have 50

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants