diff options
| author |   Julia Lawall <julia@diku.dk> | 2009-12-13 01:40:55 +0000 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2009-12-13 19:56:33 -0800 |
| commit | 710708e82bfbdd392ebdef5743e68fef0fde91a6 (patch) | |
| tree | 736538bc9cc7b186910373882e990d1670c1ae8f /drivers/atm/iphase.c | |
| parent | 3c574_cs: disable irq before calling el3_interrupt (diff) | |
| download | linux-dev-710708e82bfbdd392ebdef5743e68fef0fde91a6.tar.xz linux-dev-710708e82bfbdd392ebdef5743e68fef0fde91a6.zip | |
drivers/atm: Correct code taking the size of a pointer
sizeof(TstSchedTbl) is just the size of the pointer. Change it to the size
of the referenced data.
A simplified version of the semantic patch that finds this problem is as
follows: (http://coccinelle.lip6.fr/)
// <smpl>
@@
expression *x;
expression f;
type T;
@@
*f(...,(T)x,...)
// </smpl>
Signed-off-by: Julia Lawall <julia@diku.dk>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/atm/iphase.c')
| -rw-r--r-- | drivers/atm/iphase.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/atm/iphase.c b/drivers/atm/iphase.c index f734b345ac71..25a4c86f839b 100644 --- a/drivers/atm/iphase.c +++ b/drivers/atm/iphase.c @@ -557,7 +557,7 @@ static int ia_cbr_setup (IADEV *dev, struct atm_vcc *vcc) { memcpy((caddr_t)&cbrVC,(caddr_t)TstSchedTbl,sizeof(cbrVC)); } /* while */ // Move this VCI number into this location of the CBR Sched table. - memcpy((caddr_t)TstSchedTbl, (caddr_t)&vcIndex,sizeof(TstSchedTbl)); + memcpy((caddr_t)TstSchedTbl, (caddr_t)&vcIndex, sizeof(*TstSchedTbl)); dev->CbrRemEntries--; toBeAssigned--; } /* while */ |