aboutsummaryrefslogtreecommitdiffstats
path: root/sound/usb
diff options
context:
space:
mode:
authorDaniel Mack <zonque@gmail.com>2013-03-04 06:50:05 -0500
committerTakashi Iwai <tiwai@suse.de>2013-03-07 03:24:12 -0500
commit2dad9402192250d4061332b6a9be71ebf8493c49 (patch)
treed3331b04ee77b962361f20e49a2e491d4ebbddb4 /sound/usb
parentf1f6b8f65ff08afed4532b88de1a3bbea773787f (diff)
ALSA: snd-usb-caiaq: fix smatch warnings
Fix three smatch warnings recently introduced: sound/usb/caiaq/device.c:166 usb_ep1_command_reply_dispatch() warn: variable dereferenced before check 'cdev' (see line 163) sound/usb/caiaq/device.c:517 snd_disconnect() warn: variable dereferenced before check 'card' (see line 514) sound/usb/caiaq/input.c:510 snd_usb_caiaq_ep4_reply_dispatch() warn: variable dereferenced before check 'cdev' (see line 506) Signed-off-by: Daniel Mack <zonque@gmail.com> Reported-by: Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by: Takashi Iwai <tiwai@suse.de>
Diffstat (limited to 'sound/usb')
-rw-r--r--sound/usb/caiaq/device.c8
-rw-r--r--sound/usb/caiaq/input.c2
2 files changed, 5 insertions, 5 deletions
diff --git a/sound/usb/caiaq/device.c b/sound/usb/caiaq/device.c
index d898f737c07e..48b63ccc78c7 100644
--- a/sound/usb/caiaq/device.c
+++ b/sound/usb/caiaq/device.c
@@ -159,8 +159,8 @@ static struct usb_device_id snd_usb_id_table[] = {
159static void usb_ep1_command_reply_dispatch (struct urb* urb) 159static void usb_ep1_command_reply_dispatch (struct urb* urb)
160{ 160{
161 int ret; 161 int ret;
162 struct device *dev = &urb->dev->dev;
162 struct snd_usb_caiaqdev *cdev = urb->context; 163 struct snd_usb_caiaqdev *cdev = urb->context;
163 struct device *dev = caiaqdev_to_dev(cdev);
164 unsigned char *buf = urb->transfer_buffer; 164 unsigned char *buf = urb->transfer_buffer;
165 165
166 if (urb->status || !cdev) { 166 if (urb->status || !cdev) {
@@ -511,13 +511,13 @@ static int snd_probe(struct usb_interface *intf,
511static void snd_disconnect(struct usb_interface *intf) 511static void snd_disconnect(struct usb_interface *intf)
512{ 512{
513 struct snd_card *card = usb_get_intfdata(intf); 513 struct snd_card *card = usb_get_intfdata(intf);
514 struct snd_usb_caiaqdev *cdev = caiaqdev(card); 514 struct device *dev = intf->usb_dev;
515 struct device *dev; 515 struct snd_usb_caiaqdev *cdev;
516 516
517 if (!card) 517 if (!card)
518 return; 518 return;
519 519
520 dev = caiaqdev_to_dev(cdev); 520 cdev = caiaqdev(card);
521 dev_dbg(dev, "%s(%p)\n", __func__, intf); 521 dev_dbg(dev, "%s(%p)\n", __func__, intf);
522 522
523 snd_card_disconnect(card); 523 snd_card_disconnect(card);
diff --git a/sound/usb/caiaq/input.c b/sound/usb/caiaq/input.c
index fe8f4b4fd369..efc70ae915c5 100644
--- a/sound/usb/caiaq/input.c
+++ b/sound/usb/caiaq/input.c
@@ -503,8 +503,8 @@ static void snd_usb_caiaq_maschine_dispatch(struct snd_usb_caiaqdev *cdev,
503static void snd_usb_caiaq_ep4_reply_dispatch(struct urb *urb) 503static void snd_usb_caiaq_ep4_reply_dispatch(struct urb *urb)
504{ 504{
505 struct snd_usb_caiaqdev *cdev = urb->context; 505 struct snd_usb_caiaqdev *cdev = urb->context;
506 struct device *dev = caiaqdev_to_dev(cdev);
507 unsigned char *buf = urb->transfer_buffer; 506 unsigned char *buf = urb->transfer_buffer;
507 struct device *dev = &urb->dev->dev;
508 int ret; 508 int ret;
509 509
510 if (urb->status || !cdev || urb != cdev->ep4_in_urb) 510 if (urb->status || !cdev || urb != cdev->ep4_in_urb)