Official Arduino WiFi Shield - Issues - Improvements - Call to action

SurferTim · June 20, 2014, 12:39pm

Same problems. AFAIK, nobody is even looking into it. Here are my reports. Even after 6 months, they don't have anyone assigned to the problems.

github.com/arduino/Arduino

WiFi server uses only one socket

opened 01:22PM - 03 Dec 13 UTC

closed 01:33PM - 12 Jul 16 UTC

SurferTim

Library: Wifi

The wifi shield library apparently uses only one socket, and that one poorly. Th…e function below displays the state of each of the 4 sockets, and the port number if a server. The first number is the status of that socket. -1 = available 0 to 3 = used The second is the port. When I start the server and call this function, I get this 0 80 -1 0 -1 0 -1 0 After the first request from a client, I get this -1 80 -1 0 -1 0 -1 0 That is indicating to me the server is not starting a second socket listening when a client connects on the first as does the ethernet shield. Also note that socket 0 is now available after the first request. That seems like a bug to me. I would expect this after the first client request -1 0 1 80 -1 0 -1 0 I am getting wrong image downloads and corrupted images using the server library. I think this may be related to the use of only one socket. Or am I seeing this incorrectly? ``` void ShowSockStatus() { for(int x = 0; x < MAX_SOCK_NUM; x++) { Serial.print(WiFi._state[x]); Serial.print(" "); Serial.println(WiFi._server_port[x]); } } ```

github.com/arduino/Arduino

WiFi shield client does not release socket on connect fail (with fix)

opened 10:57AM - 05 Dec 13 UTC

closed 01:32PM - 12 Jul 16 UTC

SurferTim

Type: Bug Library: Wifi

The wifi shield client code is not releasing the socket when the connection fail…s. This results in the client using all the sockets in 4 failed attempts, and the connection will fail until the Arduino is reset/rebooted. IDE v1.0.5 and v1.5.5. Here is the fix. This is in WiFiClient.cpp ``` int WiFiClient::connect(IPAddress ip, uint16_t port) { _sock = getFirstSocket(); if (_sock != NO_SOCKET_AVAIL) { ServerDrv::startClient(uint32_t(ip), port, _sock); WiFiClass::_state[_sock] = _sock; unsigned long start = millis(); // wait 4 second for the connection to close while (!connected() && millis() - start < 10000) delay(1); if (!connected()) { // Add this WiFiClass::_state[_sock] = -1; // end of add return 0; } }else{ Serial.println("No Socket available"); return 0; } return 1; } ```

github.com/arduino/Arduino

WiFi library getSocket function incorrect

opened 11:45AM - 11 Dec 13 UTC

closed 01:30PM - 12 Jul 16 UTC

SurferTim

Type: Bug Library: Wifi

The way the getSocket() function allocates sockets is incorrect. It uses the wro…ng array to determine availability. This will cause errors if using a client connection with any of the listening protocols (udp and tcp server). This is in WiFi.cpp. IDE v1.0.5 and v1.5.5. ``` uint8_t WiFiClass::getSocket() { for (uint8_t i = 0; i < MAX_SOCK_NUM; ++i) { if (WiFiClass::_server_port[i] == 0) { return i; } } return NO_SOCKET_AVAIL; } ``` This is the function I currently use, and it works good. It uses the state array instead. This is the same variable array that the client socket allocation uses. ``` uint8_t WiFiClass::getSocket() { for (uint8_t i = 0; i < MAX_SOCK_NUM; ++i) { if (WiFiClass::_state[i] == 0) { return i; } } return NO_SOCKET_AVAIL; } ```

github.com/arduino/Arduino

Wifi client frees socket allocation on a listening socket

opened 11:55AM - 11 Dec 13 UTC

closed 01:29PM - 12 Jul 16 UTC

SurferTim

Type: Bug Library: Wifi

If using a server, this incorrectly released a socket that should still be reser…ved for the server. If the server_port array entry for that socket is non-zero, it should not release the socket. This is in WiFiClient.cpp. IDE v1.0.5 and v1.5.5. ``` void WiFiClient::stop() { if (_sock == 255) return; ServerDrv::stopClient(_sock); // add this line to prevent release of a listening socket (server) if(WiFiClass::_server_port[_sock] == 0) WiFiClass::_state[_sock] = NA_STATE; int count = 0; // wait maximum 5 secs for the connection to close while (status() != CLOSED && ++count < 50) delay(100); _sock = 255; } ```

Topic		Replies	Views
WiFi shield server library corruption/crashes Networking, Protocols, and Devices	13	4883	May 6, 2021
Official Wifi Shield Disconnect and Reconnect Problems Networking, Protocols, and Devices	15	10246	May 6, 2021
[Problem] WiFi shield stops working out of nowhere Programming	10	3303	May 5, 2021
WiFi Shield client code test Networking, Protocols, and Devices	8	3043	May 6, 2021
Ethernet shield problems! Time to update library? Troubleshooting	50	62614	May 6, 2021

Official Arduino WiFi Shield - Issues - Improvements - Call to action

Related topics