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

Mayble utp_context_stats counters needs to be uint64 type? #95

Open
freak82 opened this issue Jun 16, 2016 · 0 comments
Open

Mayble utp_context_stats counters needs to be uint64 type? #95

freak82 opened this issue Jun 16, 2016 · 0 comments

Comments

@freak82
Copy link

freak82 commented Jun 16, 2016

It seems to me that the counters in utp_context_stats needs to be bigger than uint32 (the current type).
I was playing with a seeding application using the libutp. I noticed that for a single 1.4GB file the count of the smallest received packets (<= 23 Bytes, ACKS) reached over a million. Same was true for the count of biggest sent packets (> 1400 Bytes). I can imagine that for 24/7 application these 32 bit counters will overflow in few (1-3) weeks.

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

1 participant