Building with Arduino/esp8266 2.4.0-rc2

[jake-b]

I've upgraded to 2.4.0-rc2 because it is supposed to have the KRAK WPA2 vulnerability fix.

Unfortunately, it seems like err_t may have gone from an int8_t to an int32_t -- You can see this in the diffs to the Arduino/esp8266 sdk here, where LWIP_ERR_T is defined to s32_t:

esp8266/Arduino@e049032#diff-171efba17be7978e6f6c00ab1b100ae0

err_t seems hard-coded in ESPAsyncTCP to an int32_t

I tried my hand at brute forcing the change, but so far just panics from my dev board, so I'm guessing I'm missing something somewhere. I'll continue to play and if I get something that works, I'll report back.

Update: Just a copy of the errors seems when compiling. Emphasis on where the function prototype differ.

ESPAsyncTCP.cpp: In constructor 'AsyncClient::AsyncClient(tcp_pcb*)':
ESPAsyncTCP.cpp:77:28: error: invalid conversion from
'int8_t ()(void, tcp_pcb*, pbuf*, int8_t) {aka signed char ()(void, tcp_pcb*, pbuf*, signed char)}'
to
'tcp_recv_fn {aka long int ()(void, tcp_pcb*, pbuf*, long int)}' [-fpermissive]
tcp_recv(_pcb, &_s_recv);

Other functions that also show similar errors in other places: tcp_recv, tcp_sent, tcp_err, tcp_poll, tcp_accept

[jake-b]

Nevermind. I didn't have the latest commit. You beat me to it and I'm reporting an issue you already fixed in 036ea44

I'm still getting panics, but I I think it might be some other incompatibility with my own code.