diff options
author | Mikael Pettersson <[email protected]> | 2014-05-14 16:23:02 +0200 |
---|---|---|
committer | Mikael Pettersson <[email protected]> | 2014-05-14 16:51:15 +0200 |
commit | 78540afe8d4a15fe3d6c42208b82bde546e4c0c8 (patch) | |
tree | 471855440f987e13a8e4ba9efd6ad717ed7631eb /lib/inets/test/tftp_test_lib.hrl | |
parent | 5ade234d37600ea80dbb309f431c615937ea253d (diff) | |
download | otp-78540afe8d4a15fe3d6c42208b82bde546e4c0c8.tar.gz otp-78540afe8d4a15fe3d6c42208b82bde546e4c0c8.tar.bz2 otp-78540afe8d4a15fe3d6c42208b82bde546e4c0c8.zip |
Fix efile_openfile() to handle stat() failure
If the initial stat() fails then efile_openfile() will still proceed
to open() the file. If that succeeds and the caller passed a non-NULL
pSize, then it will copy bogus data from the statbuf into *pSize. This
has been observed to cause file:read_file/1 to return truncated file
data with no error indication.
The use case involved a large file system mounted via NFS, with some
directories containing large number of files, and NFS mount options
that allow the NFS client to return EIO if the NFS server does not
respond quickly enough. Depending on the caching state of the client
and server machines, a few stat() calls (fewer than 1 per 10 million)
would take long enough to trigger EIO errors, but subsequent open()
calls would succeed, and read_file/1 would return truncated data. This
sequence of events has been observed via "strace" on beam.smp.
Signed-off-by: Mikael Pettersson <[email protected]>
Diffstat (limited to 'lib/inets/test/tftp_test_lib.hrl')
0 files changed, 0 insertions, 0 deletions