because the compiler is too stupid to notice that we check for
(ssize_t) foo < 0, and disallow it.
packet_len = fr_tacacs_length(buffer, in_buffer);
if (packet_len < 0) return -1;
- if (in_buffer < packet_len) {
+ if (in_buffer < (size_t) packet_len) {
*leftover = in_buffer;
return 0;
}
* We've read more than one packet. Tell the caller that
* there's more data available, and return only one packet.
*/
- if (in_buffer > packet_len) {
+ if (in_buffer > (size_t) packet_len) {
*leftover = in_buffer - packet_len;
}