diff options
| author | Maxim Dounin <mdounin@mdounin.ru> | 2013-02-13 14:39:46 +0000 |
|---|---|---|
| committer | Maxim Dounin <mdounin@mdounin.ru> | 2013-02-13 14:39:46 +0000 |
| commit | d2c9f4554f201bde00281d58d4de600a0e8877a1 (patch) | |
| tree | 41e7cdf2a4980cf58169bce65767c2f27f5584b3 /src | |
| parent | 2761d480a200cbbb304476ff695515c8e7481465 (diff) | |
| download | nginx-d2c9f4554f201bde00281d58d4de600a0e8877a1.tar.gz nginx-d2c9f4554f201bde00281d58d4de600a0e8877a1.zip | |
Fixed false memset warning on Linux with -O3 (ticket #275).
Prodded by John Leach.
Diffstat (limited to 'src')
| -rw-r--r-- | src/http/modules/ngx_http_autoindex_module.c | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/src/http/modules/ngx_http_autoindex_module.c b/src/http/modules/ngx_http_autoindex_module.c index 450a48e50..fb46d65d1 100644 --- a/src/http/modules/ngx_http_autoindex_module.c +++ b/src/http/modules/ngx_http_autoindex_module.c @@ -489,8 +489,11 @@ ngx_http_autoindex_handler(ngx_http_request_t *r) } b->last = ngx_cpymem(b->last, "</a>", sizeof("</a>") - 1); - ngx_memset(b->last, ' ', NGX_HTTP_AUTOINDEX_NAME_LEN - len); - b->last += NGX_HTTP_AUTOINDEX_NAME_LEN - len; + + if (NGX_HTTP_AUTOINDEX_NAME_LEN - len > 0) { + ngx_memset(b->last, ' ', NGX_HTTP_AUTOINDEX_NAME_LEN - len); + b->last += NGX_HTTP_AUTOINDEX_NAME_LEN - len; + } } *b->last++ = ' '; |