diff options
| author | Andrew Clayton <ac@sigsegv.uk> | 2025-06-26 22:20:46 +0100 |
|---|---|---|
| committer | Andrew Clayton <a.clayton@nginx.com> | 2025-06-26 22:24:01 +0100 |
| commit | 0c5bb8a5e10657cec505729596c8b2f0c794cc8f (patch) | |
| tree | 1695c1ba5698c539795bf265f56f9b606dbced23 /src/stream/ngx_stream_geo_module.c | |
| parent | 4eaecc5e8aa7bbaf9e58bf56560a8b1e67d0a8b7 (diff) | |
| download | nginx-0c5bb8a5e10657cec505729596c8b2f0c794cc8f.tar.gz nginx-0c5bb8a5e10657cec505729596c8b2f0c794cc8f.tar.bz2 | |
Remove short read checksno-short-read-checks
This removes a bunch of checks for cases where we _read_ less bytes than
asked for.
There are several reasons why we may _read_ less bytes including reading
data from a network file system.
This has proven to be an issue.
Closes: https://github.com/nginx/nginx/issues/265
Signed-off-by: Andrew Clayton <ac@sigsegv.uk>
Diffstat (limited to 'src/stream/ngx_stream_geo_module.c')
| -rw-r--r-- | src/stream/ngx_stream_geo_module.c | 7 |
1 files changed, 0 insertions, 7 deletions
diff --git a/src/stream/ngx_stream_geo_module.c b/src/stream/ngx_stream_geo_module.c index 2324bef0d..910915c20 100644 --- a/src/stream/ngx_stream_geo_module.c +++ b/src/stream/ngx_stream_geo_module.c @@ -1412,13 +1412,6 @@ ngx_stream_geo_include_binary_base(ngx_conf_t *cf, goto failed; } - if ((size_t) n != size) { - ngx_conf_log_error(NGX_LOG_CRIT, cf, 0, - ngx_read_file_n " \"%s\" returned only %z bytes instead of %z", - name->data, n, size); - goto failed; - } - header = (ngx_stream_geo_header_t *) base; if (size < 16 || ngx_memcmp(&ngx_stream_geo_header, header, 12) != 0) { |