From: Alexander Boyko Date: Tue, 28 May 2019 10:07:12 +0000 (-0400) Subject: LU-12140 lnet: adds checking msg len X-Git-Tag: 2.12.54~10 X-Git-Url: https://git.whamcloud.com/?p=fs%2Flustre-release.git;a=commitdiff_plain;h=4d43a6c3b182485ffaf7d94c726653b1a36d1b9b 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 --- 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;