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

fix nTime conversion #5

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

DeckerSU
Copy link

The first argument of parseInt should be string, not buffer.
Without this fix there is possible the case in which shares will
be rejected with ntime out of range.

Small example:

// string f7b40161 <Buffer f7 b4 01 61>
// 10 1627501815 1627501827

For nTime = "f7b40161" , nTimeInt will be equal 10, which is totally incorrect.

More details available here:

DeckerSU/node-stratum-pool@796410e

The first argument of parseInt should be string, not buffer.
Without this fix there is possible the case in which shares will
be rejected with `ntime out of range`.

Small example:

// string f7b40161 <Buffer f7 b4 01 61>
// 10 1627501815 1627501827

For nTime = "f7b40161" , nTimeInt will be equal 10, which is totally incorrect.

More details available here:

DeckerSU/node-stratum-pool@796410e
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.

1 participant