Make sure iContentLength doesn't wrap around due to malformed packets

[facchinm]

@rhpco @rhaidiz can you take a look?

[github-actions]

Memory usage change @ 1a3fb98

Board	flash	%	RAM for global variables	%
arduino:samd:mkr1000	🔺 0 - +8	0.0 - 0.0	0 - 0	0.0 - 0.0

Click for full report table

Board	examples/BasicAuthGet flash	%	examples/BasicAuthGet RAM for global variables	%	examples/CustomHeader flash	%	examples/CustomHeader RAM for global variables	%	examples/DweetGet flash	%	examples/DweetGet RAM for global variables	%	examples/DweetPost flash	%	examples/DweetPost RAM for global variables	%	examples/HueBlink flash	%	examples/HueBlink RAM for global variables	%	examples/ParseURL flash	%	examples/ParseURL RAM for global variables	%	examples/PostWithHeaders flash	%	examples/PostWithHeaders RAM for global variables	%	examples/SimpleDelete flash	%	examples/SimpleDelete RAM for global variables	%	examples/SimpleGet flash	%	examples/SimpleGet RAM for global variables	%	examples/SimpleHttpExample flash	%	examples/SimpleHttpExample RAM for global variables	%	examples/SimplePost flash	%	examples/SimplePost RAM for global variables	%	examples/SimplePut flash	%	examples/SimplePut RAM for global variables	%	examples/SimpleWebSocket flash	%	examples/SimpleWebSocket RAM for global variables	%
arduino:samd:mkr1000	0	0.0	0	0.0	0	0.0	0	0.0	8	0.0	0	0.0	0	0.0	0	0.0	8	0.0	0	0.0	0	0.0	0	0.0	0	0.0	0	0.0	0	0.0	0	0.0	8	0.0	0	0.0	8	0.0	0	0.0	8	0.0	0	0.0	8	0.0	0	0.0	8	0.0	0	0.0

Click for full report CSV

Board,examples/BasicAuthGet<br>flash,%,examples/BasicAuthGet<br>RAM for global variables,%,examples/CustomHeader<br>flash,%,examples/CustomHeader<br>RAM for global variables,%,examples/DweetGet<br>flash,%,examples/DweetGet<br>RAM for global variables,%,examples/DweetPost<br>flash,%,examples/DweetPost<br>RAM for global variables,%,examples/HueBlink<br>flash,%,examples/HueBlink<br>RAM for global variables,%,examples/ParseURL<br>flash,%,examples/ParseURL<br>RAM for global variables,%,examples/PostWithHeaders<br>flash,%,examples/PostWithHeaders<br>RAM for global variables,%,examples/SimpleDelete<br>flash,%,examples/SimpleDelete<br>RAM for global variables,%,examples/SimpleGet<br>flash,%,examples/SimpleGet<br>RAM for global variables,%,examples/SimpleHttpExample<br>flash,%,examples/SimpleHttpExample<br>RAM for global variables,%,examples/SimplePost<br>flash,%,examples/SimplePost<br>RAM for global variables,%,examples/SimplePut<br>flash,%,examples/SimplePut<br>RAM for global variables,%,examples/SimpleWebSocket<br>flash,%,examples/SimpleWebSocket<br>RAM for global variables,%
arduino:samd:mkr1000,0,0.0,0,0.0,0,0.0,0,0.0,8,0.0,0,0.0,0,0.0,0,0.0,8,0.0,0,0.0,0,0.0,0,0.0,0,0.0,0,0.0,0,0.0,0,0.0,8,0.0,0,0.0,8,0.0,0,0.0,8,0.0,0,0.0,8,0.0,0,0.0,8,0.0,0,0.0

[rhpco]

LGTM

[alrvid]

Can we assume that the incoming value is already checked for not being negative? I think this code might make the new value positive for some negative incoming values. I don't know if this matters, since I don't know if there are checks for negative values somewhere later, or if negative incoming values are even possible, or can break things at a later stage.