RFR(XXS): 8242485: Null _file checking in fileStream::flush()

[David Holmes]

Hi,

On 10/04/2020 10:34 pm, Denghui Dong wrote:
> Hi team,
> 
> Could you please review this small patch?
> 
> JBS: https://bugs.openjdk.java.net/browse/JDK-8242485
> webrev: http://cr.openjdk.java.net/~ddong/8242485/webrev.00/
> 
> I found the method "fileStream::flush()" lacks null check before fflush  which will cause
> all open output streams are flushed by fflush() if _file is NULL.

Good catch. Our fileStream abstraction seems somewhat broken when it 
comes to errors! I think fileStream::readln also needs a NULL check for 
clarity if nothing else.

Thanks,
David

> Thanks,
> Denghui Dong
>