IB/mthca: Use correct structure size in call to memset()

Roland Dreier · Roland Dreier · commit 99d4f22e91d2 · 2007-02-10T08:00:47.000-08:00
When clearing the ib_ah_attr parameter in to_ib_ah_attr(), use sizeof
*ib_ah_attr instead of sizeof *path.

Pointed out by Jack Morgenstein &lt;jackm@mellanox.co.il&gt;.

Signed-off-by: Roland Dreier &lt;rolandd@cisco.com&gt;
diff --git a/drivers/infiniband/hw/mthca/mthca_qp.c b/drivers/infiniband/hw/mthca/mthca_qp.c
@@ -399,7 +399,7 @@ static int to_ib_qp_access_flags(int mthca_flags)
 static void to_ib_ah_attr(struct mthca_dev *dev, struct ib_ah_attr *ib_ah_attr,
 				struct mthca_qp_path *path)
 {
-	memset(ib_ah_attr, 0, sizeof *path);
+	memset(ib_ah_attr, 0, sizeof *ib_ah_attr);
 	ib_ah_attr->port_num 	  = (be32_to_cpu(path->port_pkey) >> 24) & 0x3;
 
 	if (ib_ah_attr->port_num == 0 || ib_ah_attr->port_num > dev->limits.num_ports)

Original file line number	Diff line number	Diff line change
`@@ -399,7 +399,7 @@ static int to_ib_qp_access_flags(int mthca_flags)`
`399`	`399`	`static void to_ib_ah_attr(struct mthca_dev dev, struct ib_ah_attr ib_ah_attr,`
`400`	`400`	`struct mthca_qp_path *path)`
`401`	`401`	`{`
`402`		`- memset(ib_ah_attr, 0, sizeof *path);`
	`402`	`+ memset(ib_ah_attr, 0, sizeof *ib_ah_attr);`
`403`	`403`	`ib_ah_attr->port_num = (be32_to_cpu(path->port_pkey) >> 24) & 0x3;`
`404`	`404`
`405`	`405`	`if (ib_ah_attr->port_num == 0 \|\| ib_ah_attr->port_num > dev->limits.num_ports)`