Timeout rollover problem

[cwalger]

This timeout math looks like it will not work properly during a millis() rollover.
I think first two lines here should be changed to the last two lines:

//int timeout_at = millis() + _timeout;
//while (!client.available() && timeout_at - millis() < 0) {
unsigned long start = millis();
while (!client.available() && millis() - start < _timeout) {	//millis() - start < ms

[teHwHoly]

Hi!
I'm really not sure, why the existing while loop shall be working..
replaced it with the following:

unsigned long start = millis();
while (!client.available()) {
	if (millis() - start > _timeout) {
		client.stop();
		return false;
	}
}

This way it stays in the while loop, waiting for the client and returns (cancels) the routine just if timeout occurs.