Strange behavior of WiFi code

[Zeni241]

I have this code to connect to wifi.

Code: Select all

tcpip_adapter_init();
    ESP_ERROR_CHECK(esp_event_loop_init(event_handler, NULL));
    wifi_init_config_t cfg = WIFI_INIT_CONFIG_DEFAULT();
    ESP_ERROR_CHECK(esp_wifi_init(&cfg));
       ESP_ERROR_CHECK(esp_wifi_set_mode(WIFI_MODE_STA));
        char *myssid = "myssid";
        char *mypw = "mypassword";
      wifi_config_t wifi_config;
    strcpy((char *)wifi_config.sta.ssid,myssid);
    strcpy((char *)wifi_config.sta.password, mypw);
    printf("wifi_config.sta.ssid=%s\n wifi_config.sta.password=%s\n",wifi_config.sta.ssid,wifi_config.sta.password);
    ESP_ERROR_CHECK(esp_wifi_set_config(WIFI_IF_STA, &wifi_config));
    ESP_ERROR_CHECK(esp_wifi_start());

If I put this code in app_main, it works, but if I put this code in an other function, say connectwifi(), and call that function in app_main, it does not work.


Further, if I use

Code: Select all

char myssid[] = "myssid";
 char mypw[] = "mypassword";

instead of

Code: Select all

char *myssid = "myssid";
 char *mypw = "mypassword";

in the above code, it does not work even in app_main.

What could be the reason(s)?

In fact idea is to get SSID and PASSWORD from NVS each time to connect to wifi.
If fails to connect to wifi in given time, wifi in access mode is started asking for user input of SSID and PASSWORD, save it to NVS, and again connect to wifi using values from NVS.

I have miserably failed to figure out how to use user input I get in AP mode, to connect to wifi in station mode.
I can get user input in AP mode, save and retrieve it from NVS, but those values just don't work in connecting in station mode.(though those retrieved values are correct when I get their print).
Any help please?
and sorry for such a long question

[brp80000]

It looks like ESP32 after IDF-3.1 no longer has working WIFI. It's time to move on STM32 ))))

[ESP_igrr]

Zeni241: When you declare wifi_config_t structure, do not forget to set all fields to zero. Either by calling 'memset' or using an initializer:

wifi_config_t wifi_config = { };

Or

wifi_config_t wifi_config;
memset(&wifi_config, 0, sizeof(wifi_config));

[Zeni241]

Good morning.
Couldn't be a better start of morning . Thanks a lot ESP_igrr, this magic mantra wifi_config_t wifi_config = { }; solved the problem I was wrestling with many days.
Now I can use call my "wifi_connect" function (where I have placed the code to connect wifi) from any where successfully.
I can even pass this function values of SSID and password (wifi_connect(char *ssid, char *pw)).
I can also use:
char myssid[] = "myssid";
char mypw[] = "mypassword";

Thanks again to ESP_igrr.

[brp80000]

Oh, you're a fun group.

[Zeni241]

Did it worked for you?

[Ritesh]

Zeni241: When you declare wifi_config_t structure, do not forget to set all fields to zero. Either by calling 'memset' or using an initializer:

wifi_config_t wifi_config = { };

Or

wifi_config_t wifi_config;
memset(&wifi_config, 0, sizeof(wifi_config));

Yes. You are correct and we need to keep habit to memset it with zero before filling values into that.