mctpd: constify is_eid_in_bridge_pool

jk-ozlabs · jk-ozlabs · commit 29b403ba0ca6 · 2025-08-25T11:07:44.000+08:00
Guarantee that we're not altering ctx/net data.

Signed-off-by: Jeremy Kerr &lt;jk@codeconstruct.com.au&gt;
diff --git a/src/mctpd.c b/src/mctpd.c
@@ -1584,11 +1584,11 @@ static int endpoint_send_set_endpoint_id(const struct peer *peer,
 }
 
 // Checks if given EID belongs to any bridge's pool range
-static bool is_eid_in_bridge_pool(struct net *n, struct ctx *ctx,
+static bool is_eid_in_bridge_pool(const struct net *n, const struct ctx *ctx,
 				  mctp_eid_t eid)
 {
 	for (int i = ctx->dyn_eid_min; i <= eid; i++) {
-		struct peer *peer = n->peers[i];
+		const struct peer *peer = n->peers[i];
 		if (peer && peer->pool_size > 0) {
 			if (eid >= peer->pool_start &&
 			    eid < peer->pool_start + peer->pool_size) {

Original file line number	Diff line number	Diff line change
`@@ -1584,11 +1584,11 @@ static int endpoint_send_set_endpoint_id(const struct peer *peer,`
`1584`	`1584`	`}`
`1585`	`1585`
`1586`	`1586`	`// Checks if given EID belongs to any bridge's pool range`
`1587`		`-static bool is_eid_in_bridge_pool(struct net n, struct ctx ctx,`
	`1587`	`+static bool is_eid_in_bridge_pool(const struct net n, const struct ctx ctx,`
`1588`	`1588`	`mctp_eid_t eid)`
`1589`	`1589`	`{`
`1590`	`1590`	`for (int i = ctx->dyn_eid_min; i <= eid; i++) {`
`1591`		`- struct peer *peer = n->peers[i];`
	`1591`	`+ const struct peer *peer = n->peers[i];`
`1592`	`1592`	`if (peer && peer->pool_size > 0) {`
`1593`	`1593`	`if (eid >= peer->pool_start &&`
`1594`	`1594`	`eid < peer->pool_start + peer->pool_size) {`