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

Field Length Problem #32

Open
KenOn10 opened this issue Aug 7, 2018 · 1 comment
Open

Field Length Problem #32

KenOn10 opened this issue Aug 7, 2018 · 1 comment

Comments

@KenOn10
Copy link

KenOn10 commented Aug 7, 2018

This code is really helpful and performs well.

But, I had problems with DBF columns having lengths >128 characters, which are parsed wrong (i.e. column lengths appear as <0). IMHO, there is a bug in function convertBinaryToInteger() in header.js. The code uses buffer.readIntLE(0, buffer.length) but should use the unsigned version instead: buffer.readUIntLE(0, buffer.length).

Thanks,
Ken

@mightyjol
Copy link

I comfirm that changing the header.js file to use
buffer.readUIntLE(0, buffer.length)
fixes the issue

Thanks

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