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

Size overflow? #78

Closed
lukateras opened this issue Apr 9, 2018 · 4 comments
Closed

Size overflow? #78

lukateras opened this issue Apr 9, 2018 · 4 comments

Comments

@lukateras
Copy link

lukateras commented Apr 9, 2018

I'm using v0.3.1. I've started Gridsync (in debug mode) and after a while size has become negative:

screenshot_2018-04-09_06-09-43

Unfortunately, when that happened, there has been nothing out of the ordinary in the logs (only another add_updated_file entry), but I can send GPG-encrypted logs on demand.

@lukateras
Copy link
Author

lukateras commented Apr 9, 2018

It has synced one more file and now it shows 830 KB, while it has uploaded at least 2GB:

screenshot_2018-04-09_06-47-44

This persists between restarts.

@lukateras
Copy link
Author

This happened again:

screenshot_2018-04-09_11-33-46

When that happens, it starts upload all over again (judging by debugging logs).

crwood added a commit that referenced this issue Apr 10, 2018
Otherwise, since PyQt uses C++ types, this can cause overflows

Fixes #78
@crwood
Copy link
Member

crwood commented Apr 10, 2018

Hi Yegor,

Thank you for catching/reporting this!

This shouldn't happen with pure python3 ints but, alas, as it turns out, it looks like PyQt's Signals/Slots are actually using C++ types under the hood, accounting for the overflow in the UI..

Anyway, I just pushed PR 79 which fixes the issue on my end (and which I will merge as soon as CI passes..). You can expect a new release within the next couple of weeks but, in the meantime, I'd welcome you to build/install from source if you'd like or need the fix sooner.

Thanks again for the report!

@lukateras
Copy link
Author

Thank you! :-)

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

No branches or pull requests

2 participants