achristmascarl / rainfrog

🐸 a database management tui for postgres
https://crates.io/crates/rainfrog
MIT License
1.55k stars 29 forks source link

Panic: cause = "index out of bounds: the len is 65535 but the index is 65535" #60

Open tjweir opened 1 week ago

tjweir commented 1 week ago

bug description

App does not start.

technical details

system info

See crash log commit: 6e103cfadfaa51c64ec58a0319d93aae0eb39dab

crash logs

If the app panicked, the error output will include a line saying "We have generated a report file at "/var/folders/xx/yyyyyyy_yyyyyyy/Z/report-abcd-efgh-ijklmnop.toml"; please get the report file and include its contents below:

Paste below:
name = "rainfrog"
operating_system = "Mac OS 14.6.1 [64-bit]"
crate_version = "0.1.16"
explanation = """
Panic occurred in file '/Users/USERNAME/.cargo/registry/src/index.crates.io-6f17d22bba15001f/ratatui-0.28.1/src/buffer/buffer.rs' at line 385
"""
cause = "index out of bounds: the len is 65535 but the index is 65535"
method = "Panic"
backtrace = """

   0: 0x10481d8fc - core::panicking::panic_bounds_check::h8ef340b051a002e0
   1: 0x104475aa4 - ratatui::buffer::buffer::Buffer::set_style::hdccb880e33aef4dd
   2: 0x104479d98 - <ratatui::widgets::paragraph::Paragraph as ratatui::widgets::Widget>::render::he501cfa0bd971c36
   3: 0x1043d7d50 - <rainfrog::components::data::Data as rainfrog::components::Component>::draw::hdd5fef695a516765
   4: 0x104406bbc - ratatui::terminal::terminal::Terminal<B>::draw::h3b4fb84960d35f6c
   5: 0x10444236c - rainfrog::app::App::run::{{closure}}::ha9138353e1f1c6f7.12679
   6: 0x104421fb8 - tokio::runtime::runtime::Runtime::block_on::hea782340a785dae2
   7: 0x10436b544 - rainfrog::main::h3697374d71d2335a
   8: 0x1043c40a8 - std::sys::backtrace::__rust_begin_short_backtrace::hde6307b704a8568a
   9: 0x10436a054 - _main"""

steps to reproduce

Steps to reproduce the behavior. If possible, include the preceding query with sensitive information removed: I resized the terminal, the app crashed and when I attempt to restart, I get the panic.

expected behavior

I expected the app to run normally.

cli output

Well, this is embarrassing.

rainfrog had a problem and crashed. To help us diagnose the problem you can send us a crash report.

We have generated a report file at "/var/folders/c0/tp6kkgx90hx85p_s07w6vwj80000gn/T/report-3f3b05a7-3a66-4f7f-9d68-320c4605b381.toml". Submit an issue or email with the subject of "rainfrog Crash Report" and include the report as an attachment.

We take privacy seriously, and do not perform any automated error collection. In order to improve the software, we rely on people to submit reports.

Thank you kindly! The application panicked (crashed). Message: index out of bounds: the len is 65535 but the index is 65535 Location: /Users/USERNAME/.cargo/registry/src/index.crates.io-6f17d22bba15001f/ratatui-0.28.1/src/buffer/buffer.rs:385

This is a bug. Consider reporting it at https://github.com/achristmascarl/rainfrog

tjweir commented 1 week ago

Oh! Looks like there is a max size of the terminal.

If columns * rows >= 65535, we get a panic.

tjweir commented 1 week ago

https://github.com/ratatui/ratatui/issues/1286 - related, but fixed apparently.

achristmascarl commented 1 week ago

thanks for reporting; i'll investigate to see if there's any workarounds to avoid hitting ratatui's limits on larger screens

tjweir commented 1 week ago

What's the priority that is one below "low"?

:)