Commit c79fcc27 authored by Tung Nguyen's avatar Tung Nguyen Committed by Jakub Kicinski
Browse files

tipc: fix incorrect order of state message data sanity check 



When receiving a state message, function tipc_link_validate_msg()
is called to validate its header portion. Then, its data portion
is validated before it can be accessed correctly. However, current
data sanity  check is done after the message header is accessed to
update some link variables.

This commit fixes this issue by moving the data sanity check to
the beginning of state message handling and right after the header
sanity check.

Fixes: 9aa422ad ("tipc: improve size validations for received domain records")
Acked-by: default avatarJon Maloy <jmaloy@redhat.com>
Signed-off-by: default avatarTung Nguyen <tung.q.nguyen@dektech.com.au>
Link: https://lore.kernel.org/r/20220308021200.9245-1-tung.q.nguyen@dektech.com.au


Signed-off-by: default avatarJakub Kicinski <kuba@kernel.org>
parent b19ab4b3

net/tipc/link.c

+5 −4
Original line number Diff line number Diff line
@@ -2286,6 +2286,11 @@ static int tipc_link_proto_rcv(struct tipc_link *l, struct sk_buff *skb, 
		break;



	case STATE_MSG:

		/* Validate Gap ACK blocks, drop if invalid */

		glen = tipc_get_gap_ack_blks(&ga, l, hdr, true);

		if (glen > dlen)

			break;



		l->rcv_nxt_state = msg_seqno(hdr) + 1;



		/* Update own tolerance if peer indicates a non-zero value */
@@ -2311,10 +2316,6 @@ static int tipc_link_proto_rcv(struct tipc_link *l, struct sk_buff *skb, 
			break;

		}



		/* Receive Gap ACK blocks from peer if any */

		glen = tipc_get_gap_ack_blks(&ga, l, hdr, true);

		if(glen > dlen)

			break;

		tipc_mon_rcv(l->net, data + glen, dlen - glen, l->addr,

			     &l->mon_state, l->bearer_id);