lkml.org 
[lkml]   [2012]   [Dec]   [13]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 081/241] Bluetooth: Fix having bogus entries in mgmt_read_index_list reply
    Date
    3.5.7.2 -stable review patch.  If anyone has any objections, please let me know.

    ------------------

    From: Johan Hedberg <johan.hedberg@intel.com>

    commit 476e44cb19f1fbf2d5883dddcc0ce31b33b45915 upstream.

    The mgmt_read_index_list uses one loop to calculate the max needed size
    of its response with the help of an upper-bound of the controller count.
    The second loop is more strict as it checks for HCI_SETUP (which might
    have gotten set after the first loop) and could result in some indexes
    being skipped. Because of this the function needs to readjust the event
    length and index count after filling in the response array.

    Signed-off-by: Johan Hedberg <johan.hedberg@intel.com>
    Acked-by: Marcel Holtmann <marcel@holtmann.org>
    Signed-off-by: Gustavo Padovan <gustavo.padovan@collabora.co.uk>
    [ herton: unfuzz patch ]
    Signed-off-by: Herton Ronaldo Krzesinski <herton.krzesinski@canonical.com>
    ---
    net/bluetooth/mgmt.c | 11 ++++++-----
    1 file changed, 6 insertions(+), 5 deletions(-)

    diff --git a/net/bluetooth/mgmt.c b/net/bluetooth/mgmt.c
    index 3571450..ec098bd 100644
    --- a/net/bluetooth/mgmt.c
    +++ b/net/bluetooth/mgmt.c
    @@ -323,7 +323,7 @@ static int read_index_list(struct sock *sk, struct hci_dev *hdev, void *data,
    struct hci_dev *d;
    size_t rp_len;
    u16 count;
    - int i, err;
    + int err;

    BT_DBG("sock %p", sk);

    @@ -341,17 +341,18 @@ static int read_index_list(struct sock *sk, struct hci_dev *hdev, void *data,
    return -ENOMEM;
    }

    - rp->num_controllers = cpu_to_le16(count);
    -
    - i = 0;
    + count = 0;
    list_for_each_entry(d, &hci_dev_list, list) {
    if (test_bit(HCI_SETUP, &d->dev_flags))
    continue;

    - rp->index[i++] = cpu_to_le16(d->id);
    + rp->index[count++] = cpu_to_le16(d->id);
    BT_DBG("Added hci%u", d->id);
    }

    + rp->num_controllers = cpu_to_le16(count);
    + rp_len = sizeof(*rp) + (2 * count);
    +
    read_unlock(&hci_dev_list_lock);

    err = cmd_complete(sk, MGMT_INDEX_NONE, MGMT_OP_READ_INDEX_LIST, 0, rp,
    --
    1.7.9.5


    \
     
     \ /
      Last update: 2012-12-13 15:21    [W:4.097 / U:0.088 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site