RFR: 8228970: AssertionError in ResponseSubscribers$HttpResponseInputStream

[Pavel Rappo]

May I suggest using org.testng.Assert.assertThrows instead of hand-rolled
try-catch constructs in the test?

> On 6 Aug 2019, at 11:12, Patrick Concannon <patrick.concannon at oracle.com> wrote:
> 
> Hi,
> 
> Would it be possible to have my fix for JDK-8228970 reviewed?
> 
> The fix modifies the read(byte[], int, int) method from ResponseSubscribers$HttpResponseInputStream adding further checks for bad parameters. I've also added an additional test case that ensures that these checks are being met.
> 
> Further information on this bug can be found here: https://bugs.openjdk.java.net/browse/JDK-8228970
> 
> Webrev for fix: http://cr.openjdk.java.net/~dfuchs/pconcann/webrev_8228970/webrev.2/
> 
> 
> Kind regards,
> 
> Patrick
>