Issue 4503: Added check into BufferedChannel's read to avoid endless loop

[StefanoBelli]

Fix #4503

Motivation

As discussed in the referred issue, this PR aims to avoid the endless loop which will happen if the dest ByteBuf passed to the BufferedChannel's read has less remaining writable bytes than the amount of bytes required to read (read's "length" parameter)

Changes

Now, BufferedChannel.read() throws an exception if dest. buf. writableBytes is less than length

[hezhangjian]

Thanks for your contribution. Could you please add a unit test for this change?

[StefanoBelli]

Of course, I will add test cases to the existing BufferedChanelTests

[StefanoBelli]

Hi, some time has passed since this pr opened and I recall that CI error was due to unability of wget to download a bk tar (maybe some backward compat tests) compressed archive from apache.org, just saying to allow you to rerun CI and then close this PR

[StevenLuMT]

reopen's reason: rerun failure checks