Possibility of segfault on s3 filesystem

[Ukjae Jeong (Jay)]

In S3 filesystem, there are several codes that fetch environment variable and convert it to absl::string_view like the following.

• io/tensorflow_io/core/filesystems/s3/s3_filesystem.cc

  Line 193 in 8b87c3b

   if (absl::SimpleAtoi(getenv("S3_DISABLE_MULTI_PART_DOWNLOAD"), &temp_value)) 

• io/tensorflow_io/core/filesystems/s3/s3_filesystem.cc

  Line 148 in 8b87c3b

   absl::SimpleAtoi(getenv("S3_CONNECT_TIMEOUT_MSEC"), &timeout) 

• io/tensorflow_io/core/filesystems/s3/s3_filesystem.cc

  Line 152 in 8b87c3b

   absl::SimpleAtoi(getenv("S3_REQUEST_TIMEOUT_MSEC"), &timeout) 

But these codes can throw segfault since getenv can return null and the following constructor of the string_view is checking the length of the parameter (nullptr): https://github.com/abseil/abseil-cpp/blob/4f3eff442191e7e2ec2d547c9331fad256bf96d1/absl/strings/string_view.h#L193-L198

Here's the small reproduction code that just calls getenv, constructs string_view and calls SimpleAtoi sequentially: https://github.com/jeongukjae/tensorflow-io-issue-1828

As you can see the readme of the reproduction repo and main.cc, it throw segfault when converting the result of getenv(null const char*) into string_view.