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

Crash when combining an empty texture #14449

Closed
jeremyshannon opened this issue Mar 10, 2024 · 1 comment
Closed

Crash when combining an empty texture #14449

jeremyshannon opened this issue Mar 10, 2024 · 1 comment
Labels
Bug Issues that were confirmed to be a bug good first issue

Comments

@jeremyshannon
Copy link
Contributor

jeremyshannon commented Mar 10, 2024

Minetest version

Minetest 5.9.0-dev-debug-f596c36f4

Irrlicht device

No response

Operating system and version

Devuan Daedalus (eq to Debian Bookworm)

CPU model

No response

GPU model

No response

Active renderer

No response

Summary

Just tracked down a crash in Exile that my team discovered when running MT 590dev, a crash which gave this fairly unhelpful output:
/usr/include/c++/12/string_view:239: constexpr const std::basic_string_view<_CharT, _Traits>::value_type& std::basic_string_view<_CharT, _Traits>::operator[](size_type) const [with _CharT = char; _Traits = std::char_traits<char>; const_reference = const char&; size_type = long unsigned int]: Assertion '__pos < this->_M_len' failed.

A bit of gdb work and it turns out we had some code doing the very dumb thing you see here:
[combine:128x64:0,0=:32,32=cloth_upper_underwear_default.png:0,32=cloth_lower_underwear_default.png
where 0,0=: is an empty texture. I've fixed that now, but there may be other mods out there that will trip this in the future , and it'd probably be polite if the crash output was clearer about what went wrong.

Steps to reproduce

Attempt to run Exile v0.3.16 with current git, or just add a borked "[combine" modifier to whatever object or node you feel like.

@jeremyshannon jeremyshannon added the Unconfirmed bug Bug report that has not been confirmed to exist/be reproducible label Mar 10, 2024
@sfan5 sfan5 added Bug Issues that were confirmed to be a bug good first issue and removed Unconfirmed bug Bug report that has not been confirmed to exist/be reproducible labels Mar 10, 2024
@SmallJoker
Copy link
Member

02a893d

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Issues that were confirmed to be a bug good first issue
Projects
None yet
Development

No branches or pull requests

3 participants