diff options
author | Julia Lawall <julia@diku.dk> | 2009-12-12 20:40:55 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2009-12-13 22:56:33 -0500 |
commit | 710708e82bfbdd392ebdef5743e68fef0fde91a6 (patch) | |
tree | 736538bc9cc7b186910373882e990d1670c1ae8f /drivers | |
parent | 671c8806c25d06407e348690c09d7c37521eda32 (diff) |
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')
-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) { | |||
557 | memcpy((caddr_t)&cbrVC,(caddr_t)TstSchedTbl,sizeof(cbrVC)); | 557 | memcpy((caddr_t)&cbrVC,(caddr_t)TstSchedTbl,sizeof(cbrVC)); |
558 | } /* while */ | 558 | } /* while */ |
559 | // Move this VCI number into this location of the CBR Sched table. | 559 | // Move this VCI number into this location of the CBR Sched table. |
560 | memcpy((caddr_t)TstSchedTbl, (caddr_t)&vcIndex,sizeof(TstSchedTbl)); | 560 | memcpy((caddr_t)TstSchedTbl, (caddr_t)&vcIndex, sizeof(*TstSchedTbl)); |
561 | dev->CbrRemEntries--; | 561 | dev->CbrRemEntries--; |
562 | toBeAssigned--; | 562 | toBeAssigned--; |
563 | } /* while */ | 563 | } /* while */ |