search

ejeklint / ArduinoWebsocketServer

171 stars 273 forks source link

w5100 buffer overflow protection #19

Closed WildOrangutan closed 9 years ago

WildOrangutan commented 9 years ago

Hi! While working on my high speed server application, which is sending packets as fast as possible, I've often got slapped by w5100 TX buffer overflow. Solution is to check "W5100.getTXFreeSize(_sock);" before sending.

Solution source: http://www.toptal.com/c/how-i-made-a-fully-functional-arduino-weather-station-for-300

danieloneill commented 9 years ago

That's a bummer bug, but this library isn't w5100 specific, or even specific to the vanilla ethernet library.

It seems to me that this is a bug for "those guys", or do you have any suggestion for this library that would make sense with respect to this bug?

WildOrangutan commented 9 years ago

Maybe Its indeed to early to open an issue. Library may need minor fixes to examples in future.