MortalreminderPT / edit-xlsx

A quick and easy-to-use Rust library for Excel file editing.
https://crates.io/crates/edit-xlsx
25 stars 4 forks source link

Stack overflow on reading an XLSX file #21

Open FlixCoder opened 5 months ago

FlixCoder commented 5 months ago

Just calling Workbook::from_path leads to a stack overflow.. :D

Unfortunately I cannot provide the example file, but it seems to happen inside of the derived Deserialize implementation of WorkSheet, so I suspect that the fields of this type maybe consume too much space inside so the stack is overflowed? Not sure, difficult to debug..

I can also say, that when I deleted some worksheets from the Excel file containing advanced Excel features and pictures(?), leaving only a basic table as the only worksheet, it started working.

MortalreminderPT commented 5 months ago

Maybe I didn't notice the default values for some of the fields. This issue may be more difficult to fix. I used to think I tested with complex enough xlsx files, now it seems these xlsx are not rich and complex enough😿