mfg_test bug in uart setup report

Tip / Sign in to post questions, reply, level up, and achieve exciting badges. Know more

cross mob
Anonymous
Not applicable

WICED Studio Version: 5.0.1 , 6.0

When I try to modify stdio_uart to wiced_uart2 in BCM94307WCD2, the serial port fails.

So I trace the setup process and find out a bug in function  "rwl_transport_setup" in file wlu_server_shared.c In folder 43xxx_Wi-Fi\libraries\test\wl_tool\43909B0\wl\exe .

The debug action in line 156 makes the port number changed, and *argv is not a valid pointer that it should not be converted into "int*"  and reused again.

LINE156: (void)*argv++;

LINE157:  DPRINT_DBG(OUTPUT, "Port name is %s\n", *argv);

LINE158:  transport_descriptor = *(int*) rwl_open_transport(remote_type, *argv, 0, 0);

The fixes are as follows:

(Comment out line 156, 157 and treat the return value of rwl_open_transport as an interger.)

LINE156: //(void)*argv++;

LINE157: // DPRINT_DBG(OUTPUT, "Port name is %s\n", *argv);

LINE158:  transport_descriptor = (int) rwl_open_transport(remote_type, *argv, 0, 0);

Thank you for your attention.

0 Likes
0 Replies