[media] anysee: do not use buffers from stack for usb_bulk_msg()
authorAntti Palosaari <crope@iki.fi>
Tue, 26 Feb 2013 17:13:41 +0000 (14:13 -0300)
committerMauro Carvalho Chehab <mchehab@redhat.com>
Mon, 18 Mar 2013 19:54:26 +0000 (16:54 -0300)
Signed-off-by: Antti Palosaari <crope@iki.fi>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
drivers/media/usb/dvb-usb-v2/anysee.c
drivers/media/usb/dvb-usb-v2/anysee.h

index a20d691d0b639cc08baa6480d4cda99d5ce9c2c1..85ba24650f0104b538c48bd5ad8387e1a53ccb7b 100644 (file)
 #include "cxd2820r.h"
 
 DVB_DEFINE_MOD_OPT_ADAPTER_NR(adapter_nr);
-static DEFINE_MUTEX(anysee_usb_mutex);
 
 static int anysee_ctrl_msg(struct dvb_usb_device *d, u8 *sbuf, u8 slen,
        u8 *rbuf, u8 rlen)
 {
        struct anysee_state *state = d_to_priv(d);
        int act_len, ret, i;
-       u8 buf[64];
 
-       memcpy(&buf[0], sbuf, slen);
-       buf[60] = state->seq++;
+       mutex_lock(&d->usb_mutex);
 
-       mutex_lock(&anysee_usb_mutex);
+       memcpy(&state->buf[0], sbuf, slen);
+       state->buf[60] = state->seq++;
 
-       dev_dbg(&d->udev->dev, "%s: >>> %*ph\n", __func__, slen, buf);
+       dev_dbg(&d->udev->dev, "%s: >>> %*ph\n", __func__, slen, state->buf);
 
        /* We need receive one message more after dvb_usb_generic_rw due
           to weird transaction flow, which is 1 x send + 2 x receive. */
-       ret = dvb_usbv2_generic_rw(d, buf, sizeof(buf), buf, sizeof(buf));
+       ret = dvb_usbv2_generic_rw_locked(d, state->buf, sizeof(state->buf),
+                       state->buf, sizeof(state->buf));
        if (ret)
                goto error_unlock;
 
@@ -82,17 +81,16 @@ static int anysee_ctrl_msg(struct dvb_usb_device *d, u8 *sbuf, u8 slen,
        for (i = 0; i < 3; i++) {
                /* receive 2nd answer */
                ret = usb_bulk_msg(d->udev, usb_rcvbulkpipe(d->udev,
-                       d->props->generic_bulk_ctrl_endpoint), buf, sizeof(buf),
-                       &act_len, 2000);
-
+                               d->props->generic_bulk_ctrl_endpoint),
+                               state->buf, sizeof(state->buf), &act_len, 2000);
                if (ret) {
                        dev_dbg(&d->udev->dev, "%s: recv bulk message " \
                                        "failed=%d\n", __func__, ret);
                } else {
                        dev_dbg(&d->udev->dev, "%s: <<< %*ph\n", __func__,
-                                       rlen, buf);
+                                       rlen, state->buf);
 
-                       if (buf[63] != 0x4f)
+                       if (state->buf[63] != 0x4f)
                                dev_dbg(&d->udev->dev, "%s: cmd failed\n",
                                                __func__);
 
@@ -109,11 +107,10 @@ static int anysee_ctrl_msg(struct dvb_usb_device *d, u8 *sbuf, u8 slen,
 
        /* read request, copy returned data to return buf */
        if (rbuf && rlen)
-               memcpy(rbuf, buf, rlen);
+               memcpy(rbuf, state->buf, rlen);
 
 error_unlock:
-       mutex_unlock(&anysee_usb_mutex);
-
+       mutex_unlock(&d->usb_mutex);
        return ret;
 }
 
index c1a4273f14ff8075f4e881fca7243b8197d4cc1a..8f426d9fc6e1110b14c521deb770e35bbc837843 100644 (file)
@@ -52,8 +52,9 @@ enum cmd {
 };
 
 struct anysee_state {
-       u8 hw; /* PCB ID */
+       u8 buf[64];
        u8 seq;
+       u8 hw; /* PCB ID */
        u8 fe_id:1; /* frondend ID */
        u8 has_ci:1;
        u8 ci_attached:1;