Bluetooth: btbcm: Drop upper nibble version check from btbcm_initialize()

btbcm_initialize() must either return an error; or fill the passed in
fw_name, otherwise we end up passing uninitialized stack memory to
request_firmware().

Since we have a fallback hw_name of "BCM" not having a known version
in the subver field does not matter, drop the check so that we always
fill the passed in fw_name.

Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Marcel Holtmann <marcel@holtmann.org>

diff --git a/drivers/bluetooth/btbcm.c b/drivers/bluetooth/btbcm.c
index 1f498f358f607e59348c5500c2ba34725bb06900..b9e1fe0521484ed2d1cab9b015c42a52615d59f4 100644 (file)
--- a/drivers/bluetooth/btbcm.c
+++ b/drivers/bluetooth/btbcm.c
@@ -440,10 +440,6 @@ int btbcm_initialize(struct hci_dev *hdev, char *fw_name, size_t len,
                        return err;
        }
 
-       /* Upper nibble of rev should be between 0 and 3? */
-       if (((rev & 0xf000) >> 12) > 3)
-               return 0;
-
        bcm_subver_table = (hdev->bus == HCI_USB) ? bcm_usb_subver_table :
                                                    bcm_uart_subver_table;