GUVWAF

I observed this also happens on nRF52 platforms when using the MQTT client proxy. To me it looks like a stack overflow due to recursive calls in the JSON library:...

The callstack I got from the debugger is as follows: > JSONValue::StringifyImpl[abi:cxx11](unsigned int) const@0x00051908 (/firmware/src/mqtt/JSONValue.cpp:808) std::char_traits::copy@0x00037386 (/.platformio/packages/[email protected]/arm-none-eabi/include/c++/7.2.1/bits/char_traits.h:292) std::__cxx11::basic_string::_S_copy@0x00037386 (/.platformio/packages/[email protected]/arm-none-eabi/include/c++/7.2.1/bits/basic_string.h:324) std::__cxx11::basic_string::_S_copy_chars@0x00037386 (/.platformio/packages/[email protected]/arm-none-eabi/include/c++/7.2.1/bits/basic_string.h:366) std::__cxx11::basic_string::_M_construct@0x00037386 (/.platformio/packages/[email protected]/arm-none-eabi/include/c++/7.2.1/bits/basic_string.tcc:225) std::__cxx11::basic_string::_M_construct_aux@0x00051b3a (/.platformio/packages/[email protected]/arm-none-eabi/include/c++/7.2.1/bits/basic_string.h:220) std::__cxx11::basic_string::_M_construct@0x00051b3a (/.platformio/packages/[email protected]/arm-none-eabi/include/c++/7.2.1/bits/basic_string.h:239) std::__cxx11::basic_string::basic_string@0x00051b3a (/.platformio/packages/[email protected]/arm-none-eabi/include/c++/7.2.1/bits/basic_string.h:424) std::pair::pair@0x00051b3a...

@arne182 I just tried this to be sure, but as I expected this doesn’t help because the memory allocation is for `typedef std::map JSONObject;`. I believe all `std::string`s in the...

That sounds like an answer generated by an LLM. If you can let it write the required code as well, that’d be great because it would be a lot of...

@Marx1 If you’re depending on JSON for nRF52 boards, I would recommend to decode the protobufs yourself instead of on the device, as this issue seems not easy to resolve....

> I am not a statistician so I asked ChatGPT about the probability that the last byte of two nodes match. It came up with the result of 16.31%. Considering...

I pushed a new commit to resolve @loodydo’s comments. Also added the hop limit setting of the original transmitter to the header flags (using 3 of the 4 currently unused...

I think this PR is ready to merge, except that it’s a breaking change unfortunately. I did a test and it seems that the two unused bytes are not set...

@roha-github What do you mean with "whole name represented by UINT_64" and which probability are you referring to?

The NodeNum is only 32 bits and based on the MAC address of the device. Assuming that that number is uniformly distributed I don't get why using modulo of the...