Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1502585 > unrolled thread

[PATCH -next] staging: rts5208: rtsx.c: Fix invalid use of sizeof in rtsx_probe()

Started byWei Yongjun <weiyj.lk@gmail.com>
First post2016-10-18 01:40 +0200
Last post2016-10-18 01:40 +0200
Articles 1 — 1 participant

Back to article view | Back to linux.kernel


Contents

  [PATCH -next] staging: rts5208: rtsx.c: Fix invalid use of sizeof in rtsx_probe() Wei Yongjun <weiyj.lk@gmail.com> - 2016-10-18 01:40 +0200

#1502585 — [PATCH -next] staging: rts5208: rtsx.c: Fix invalid use of sizeof in rtsx_probe()

FromWei Yongjun <weiyj.lk@gmail.com>
Date2016-10-18 01:40 +0200
Subject[PATCH -next] staging: rts5208: rtsx.c: Fix invalid use of sizeof in rtsx_probe()
Message-ID<stpFD-2bU-9@gated-at.bofh.it>
From: Wei Yongjun <weiyongjun1@huawei.com>

sizeof() when applied to a pointer typed expression gives the
size of the pointer, not that of the pointed data.

Fixes: 2eb9d8cbb3c3 ("staging: rts5208: rtsx.c: Alloc sizeof struct")
Signed-off-by: Wei Yongjun <weiyongjun1@huawei.com>
---
 drivers/staging/rts5208/rtsx.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/staging/rts5208/rtsx.c b/drivers/staging/rts5208/rtsx.c
index f3e5efd..68d75d0 100644
--- a/drivers/staging/rts5208/rtsx.c
+++ b/drivers/staging/rts5208/rtsx.c
@@ -884,7 +884,7 @@ static int rtsx_probe(struct pci_dev *pci,
 	dev = host_to_rtsx(host);
 	memset(dev, 0, sizeof(struct rtsx_dev));
 
-	dev->chip = kzalloc(sizeof(dev->chip), GFP_KERNEL);
+	dev->chip = kzalloc(sizeof(*dev->chip), GFP_KERNEL);
 	if (!dev->chip) {
 		err = -ENOMEM;
 		goto errout;

[toc] | [standalone]


Back to top | Article view | linux.kernel


csiph-web