Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 4a45b7d4 authored Jul 01, 2006 by Bryan O'Sullivan Committed by Linus Torvalds Jul 01, 2006

[PATCH] IB/ipath: don't allow resources to be created with illegal values



Signed-off-by: Robert Walsh <robert.walsh@qlogic.com>
Signed-off-by: Bryan O'Sullivan <bryan.osullivan@qlogic.com>
Cc: "Michael S. Tsirkin" <mst@mellanox.co.il>
Cc: Roland Dreier <rolandd@cisco.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>

parent 6665ddee

drivers/infiniband/hw/ipath/ipath_mr.c

+5 −0

Original line number	Diff line number	Diff line
		@@ -170,6 +170,11 @@ struct ib_mr ipath_reg_user_mr(struct ib_pd pd, struct ib_umem *region,
		int n, m, i;
		struct ib_mr *ret;

		if (region->length == 0) {
		ret = ERR_PTR(-EINVAL);
		goto bail;
		}

		n = 0;
		list_for_each_entry(chunk, &region->chunk_list, list)
		n += chunk->nents;

drivers/infiniband/hw/ipath/ipath_qp.c

+8 −0

Original line number	Diff line number	Diff line
		@@ -667,6 +667,14 @@ struct ib_qp ipath_create_qp(struct ib_pd ibpd,
		goto bail;
		}

		if (init_attr->cap.max_send_sge +
		init_attr->cap.max_recv_sge +
		init_attr->cap.max_send_wr +
		init_attr->cap.max_recv_wr == 0) {
		ret = ERR_PTR(-EINVAL);
		goto bail;
		}

		switch (init_attr->qp_type) {
		case IB_QPT_UC:
		case IB_QPT_RC:

drivers/infiniband/hw/ipath/ipath_verbs.c

+11 −0

Original line number	Diff line number	Diff line
		@@ -792,6 +792,17 @@ static struct ib_ah ipath_create_ah(struct ib_pd pd,
		goto bail;
		}

		if (ah_attr->dlid == 0) {
		ret = ERR_PTR(-EINVAL);
		goto bail;
		}

		if (ah_attr->port_num != 1 \|\|
		ah_attr->port_num > pd->device->phys_port_cnt) {
		ret = ERR_PTR(-EINVAL);
		goto bail;
		}

		ah = kmalloc(sizeof *ah, GFP_ATOMIC);
		if (!ah) {
		ret = ERR_PTR(-ENOMEM);