From 4d43a6c3b182485ffaf7d94c726653b1a36d1b9b Mon Sep 17 00:00:00 2001 From: Alexander Boyko Date: Tue, 28 May 2019 06:07:12 -0400 Subject: [PATCH 1/1] LU-12140 lnet: adds checking msg len The LNET can't handle a msg with len larger than LNET_MTU. The next error occured for DOM 1MB LNetError: 3137:0:(lib-move.c:4143:lnet_parse()) 192.168.8.1@tcp, src 192.168.8.1@tcp: bad PUT payload 1051832 (1048576 max expected) The patch adds fragment size check. Signed-off-by: Alexander Boyko Cray-bug-id: LUS-7174 Change-Id: Id2d21ebd87ab0bf3a9114548900fab99b278ffb0 Reviewed-on: https://review.whamcloud.com/34975 Tested-by: Jenkins Reviewed-by: Alexey Lyashkov Tested-by: Maloo Reviewed-by: Mike Pershin Reviewed-by: Oleg Drokin --- lnet/lnet/lib-md.c | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/lnet/lnet/lib-md.c b/lnet/lnet/lib-md.c index 9bf890c..7737744 100644 --- a/lnet/lnet/lib-md.c +++ b/lnet/lnet/lib-md.c @@ -335,6 +335,10 @@ lnet_md_validate(struct lnet_md *umd) CERROR("Invalid option: too many fragments %u, %d max\n", umd->length, LNET_MAX_IOV); return -EINVAL; + } else if (umd->length > LNET_MTU) { + CERROR("Invalid length: too big fragment size %u, %d max\n", + umd->length, LNET_MTU); + return -EINVAL; } return 0; -- 1.8.3.1