Nginx-ru mailing list archive (nginx-ru@sysoev.ru)
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: nginx-0.7.4
On Mon, Jun 30, 2008 at 10:47:10PM +0400, proforg wrote:
> On 30.06.2008, at 16:43, Igor Sysoev wrote:
>
> >Изменения в nginx 0.7.4
> >30.06.2008
> >
> > *) Добавление: директива access_log поддерживает переменные.
> >
> > *) Добавление: директива open_log_file_cache.
> >
> > *) Добавление: ключ -g.
> >
> > *) Добавление: поддержка строки "Expect" в заголовке запроса.
> >
> > *) Исправление: большие включения в SSI могли передавались не
> >полностью.
>
>
> Проблема с autoindex всё так же актуальна :)
Эта проблема была всегда. Сейчас она появилась, потому что charset utf-8,
а имена файлов - koi8-r. Прилагаемый патч реашает эту проблему, но заставить
браузер показать кривое utf-8 имя он не в силах.
--
Игорь Сысоев
http://sysoev.ru
Index: src/http/modules/ngx_http_autoindex_module.c
===================================================================
--- src/http/modules/ngx_http_autoindex_module.c (revision 1437)
+++ src/http/modules/ngx_http_autoindex_module.c (working copy)
@@ -135,7 +135,7 @@
{
u_char *last, *filename, scale;
off_t length;
- size_t len, copy, allocated, root;
+ size_t len, utf_len, allocated, root;
ngx_tm_t tm;
ngx_err_t err;
ngx_buf_t *b;
@@ -412,15 +412,16 @@
len = entry[i].utf_len;
- if (entry[i].name.len - len) {
+ if (entry[i].name.len != len) {
if (len > NGX_HTTP_AUTOINDEX_NAME_LEN) {
- copy = NGX_HTTP_AUTOINDEX_NAME_LEN - 3 + 1;
+ utf_len = NGX_HTTP_AUTOINDEX_NAME_LEN - 3 + 1;
} else {
- copy = NGX_HTTP_AUTOINDEX_NAME_LEN + 1;
+ utf_len = NGX_HTTP_AUTOINDEX_NAME_LEN + 1;
}
- b->last = ngx_utf_cpystrn(b->last, entry[i].name.data, copy);
+ b->last = ngx_utf_cpystrn(b->last, entry[i].name.data,
+ utf_len, entry[i].name.len + 1);
last = b->last;
} else {
Index: src/core/ngx_string.c
===================================================================
--- src/core/ngx_string.c (revision 1437)
+++ src/core/ngx_string.c (working copy)
@@ -955,7 +955,7 @@
* ngx_utf_decode() decodes two and more bytes UTF sequences only
* the return values:
* 0x80 - 0x10ffff valid character
- * 0x10ffff - 0xfffffffd invalid sequence
+ * 0x110000 - 0xfffffffd invalid sequence
* 0xfffffffe incomplete sequence
* 0xffffffff error
*/
@@ -1020,29 +1020,24 @@
size_t
ngx_utf_length(u_char *p, size_t n)
{
- u_char c;
- size_t len;
- ngx_uint_t i;
+ u_char c, *last;
+ size_t len;
- for (len = 0, i = 0; i < n; len++, i++) {
+ last = p + n;
- c = p[i];
+ for (len = 0; p < last; len++) {
+ c = *p;
+
if (c < 0x80) {
+ p++;
continue;
}
- if (c >= 0xc0) {
- for (c <<= 1; c & 0x80; c <<= 1) {
- i++;
- }
-
- continue;
+ if (ngx_utf_decode(&p, n) > 0x10ffff) {
+ /* invalid utf */
+ return n;
}
-
- /* invalid utf */
-
- return n;
}
return len;
@@ -1050,36 +1045,45 @@
u_char *
-ngx_utf_cpystrn(u_char *dst, u_char *src, size_t n)
+ngx_utf_cpystrn(u_char *dst, u_char *src, size_t n, size_t len)
{
- u_char c;
+ u_char c, *next;
if (n == 0) {
return dst;
}
- for ( /* void */ ; --n; dst++, src++) {
+ while (--n) {
c = *src;
*dst = c;
if (c < 0x80) {
- if (*dst != '\0') {
+
+ if (c != '\0') {
+ dst++;
+ src++;
+ len--;
+
continue;
}
return dst;
}
- if (c >= 0xc0) {
- for (c <<= 1; c & 0x80; c <<= 1) {
- *++dst = *++src;
- }
+ next = src;
- continue;
+ if (ngx_utf_decode(&next, len) > 0x10ffff) {
+ /* invalid utf */
+ break;
}
- /* invalid utf */
+ len--;
+
+ while (src < next) {
+ *++dst = *++src;
+ len--;
+ }
}
*dst = '\0';
Index: src/core/ngx_string.h
===================================================================
--- src/core/ngx_string.h (revision 1437)
+++ src/core/ngx_string.h (working copy)
@@ -153,7 +153,7 @@
uint32_t ngx_utf_decode(u_char **p, size_t n);
size_t ngx_utf_length(u_char *p, size_t n);
-u_char *ngx_utf_cpystrn(u_char *dst, u_char *src, size_t n);
+u_char *ngx_utf_cpystrn(u_char *dst, u_char *src, size_t n, size_t len);
#define NGX_ESCAPE_URI 0
|