upgt: add some overflow checks

[avos]

Summary

In upgt_eeprom_parse():

• change type of 'option_len' from uint16_t to uint32_t to prevent multiplication overflow;
• check for 'eeprom_option' overflow (maybe useless?).

In upgt_fw_verify():

• fix Boot Record Area presence check;
• reject firmware if 'bra_option_len' overflows.

Test Plan

Untested

[• hselasky]

sys/dev/usb/wlan/if_upgt.c

1382–1383

missing range checks for eeprom_len being valid.

1412

missing range check for header->header1.len being valid.

1447

uint32_t pkglen

1462–1469

if (pkglen > (MCLBYTES - ETHER_ALIGN - IEEE80211_CRC_LEN ))

xxx error;

1470–1471

Remove this KASSERT() - we don't want to panic on corrupt USB data!

1727

Please add checks here for frame_start wrapping around.

1772–1774

The for-loop condition should read: (offset + sizeof(*uc)) <= fw->datasize

Else you might read data which is not inside the BRA.

1779

Same here.

1804

Please also add this check:

if (sizeof(struct upgt_fw_bra_option) > (size_t)(fw->datasize - offset) ||
    (bra_option_len > (size_t)(fw->datasize - offset - sizeof(struct upgt_fw_bra_option)))

[avos]

sys/dev/usb/wlan/if_upgt.c

2359

IMHO, this code path will never work: typically error handling duplicates USB_ST_TRANSFERRED event with non-zero status code passed to ieee80211_tx_complete().
Here 1) mbuf is not freed - memory leak? 2) data buffer is not moved from active to inactive ring, so Tx will stop.

[avos]

Turn some KASSERT's into checks + add more size checks.

[avos]

Add memory wrap check.

[avos]

Add some checks against actlen.