search

openthread / ot-esp32

BSD 3-Clause "New" or "Revised" License
56 stars 10 forks source link

[cli] scan command without thread configuration will crash the system #6

Closed lanyuwen closed 4 years ago

lanyuwen commented 4 years ago

scan command without thread configuration will crash the system, no problem if using it after thread interface gets up.

wgtdkp commented 4 years ago

This is a bug in OpenThread RadioSpinel implementation. PR https://github.com/openthread/openthread/pull/5068 is submitted, you may checkout it and test at your local.

I will later update the openthread submodule once the PR got merged.

jwhui commented 4 years ago

https://github.com/openthread/openthread/pull/5068 has been merged.

wgtdkp commented 4 years ago

openthread/openthread#5068 has been merged.

will fire a PR after testing on real device with latest openthread tomorrow.

lanyuwen commented 4 years ago

it won't crash anymore, however, still no output until thread interface is up.

wgtdkp commented 4 years ago

@lanyuwen

it won't crash anymore, however, still no output until thread interface is up.

the scan command scans for existing Thread Networks, is there already a Thread Network that is reachable from the ESP32 node?

lanyuwen commented 4 years ago

I can get some results after issuing "ifconfig up", or "ifconfig up" and "ifconfig down". However, I got nothing unless these commands executed.

I don't think this behavior is expected. But I think this is a low priority problem and can be tracked by a separated issue. Let's close this one for now.

lanyuwen commented 4 years ago

reopened to track the scan failure fix progress