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

memory allocation error when reading xls files with merged regions #444

Open
bwkam opened this issue Jun 24, 2024 · 3 comments
Open

memory allocation error when reading xls files with merged regions #444

bwkam opened this issue Jun 24, 2024 · 3 comments
Labels

Comments

@bwkam
Copy link

bwkam commented Jun 24, 2024

Hello, I was trying out the new merged regions commits recently, it all works perfectly fine except for XLS files. I'm not sure why. I've managed to create a minimal reproducible sample of the bug. When I run it, it panics with:

2024-06-24T11:19:14.794480Z DEBUG calamine::cfb: load minis    
memory allocation of 2199023255552 bytes failed

This is the sample: https://github.com/bwkam/calamine-bug-sample
I checked, and this is also reproducible with v0.25.

Edit: I tried with another XLS file, and it worked fine. I don't know what's wrong with this one. Perhaps its something with the XLSX to XLS converter I used.

Thanks.

@tafia
Copy link
Owner

tafia commented Jul 8, 2024

Hey, sorry for the late response. I can't access the link, do you mind uploading it again?

@bwkam
Copy link
Author

bwkam commented Jul 8, 2024

Sorry, forgot it was private. Should work now

@tafia
Copy link
Owner

tafia commented Jul 8, 2024

Ok I can reproduce. It's late where I live I'll try to have a look ASAP.

@tafia tafia added the bug label Jul 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants