0 Replies Latest reply on Nov 15, 2017 5:48 PM by joeccmou_2842441

    mfg_test bug in uart setup report

    joeccmou_2842441

      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.