[Coverity CID: 240703] Improper use of negative value in drivers/bluetooth/hci/h4.c

[zephyrbot]

Static code scan issues found in file:

https://github.com/zephyrproject-rtos/zephyr/tree/c0fcd35531611bbe35376c62a9e50744d6904940/drivers/bluetooth/hci/h4.c#L294

Category: Integer handling issues
Function: read_payload
Component: Bluetooth
CID: 240703

Details:

zephyr/drivers/bluetooth/hci/h4.c

Line 294 in c0fcd35

net_buf_add(rx.buf, read);

288              }
289     
290              copy_hdr(rx.buf);
291      }
292     
293      read = uart_fifo_read(h4_dev, net_buf_tail(rx.buf), rx.remaining);
>>>     CID 240703:  Integer handling issues  (NEGATIVE_RETURNS)
>>>     "read" is passed to a parameter that cannot be negative.
294      net_buf_add(rx.buf, read);
295      rx.remaining -= read;
296     
297      BT_DBG("got %d bytes, remaining %u", read, rx.remaining);
298      BT_DBG("Payload (len %u): %s", rx.buf->len,
299             bt_hex(rx.buf->data, rx.buf->len));

For more information about the violation, check the Coverity Reference. (CWE-394)

Please fix or provide comments in coverity using the link:

https://scan9.coverity.com/reports.htm#v29271/p12996

Note: This issue was created automatically. Priority was set based on classification
of the file affected and the impact field in coverity. Assignees were set using the CODEOWNERS file.