Nginx-ru mailing list archive (nginx-ru@sysoev.ru)
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: nginx-0.8.30
Hello!
On Mon, Dec 21, 2009 at 03:50:27PM +0300, Igor Sysoev wrote:
> On Mon, Dec 21, 2009 at 02:36:26PM +0200, Andrew Sitnikov wrote:
>
> > Hello Igor,
> >
> > IS> Это не только gcc 4.1, но на фрибсдшном наборе gcc это не
> > воспроизводится.
> > IS> Твой вариант с memcpy мне не нравится: оно не мешает при
> > переконфигурации,
> > IS> но как общее решение не подходит - зачем копировать лишнее в run-time.
> > на 0.7.x такого кстати нет.
>
> Тут две проблемы в разных файлах.
> Твоя появилась в 0.8.12.
> А та, которую фиксил Макс, скорее всего, в 0.8.21.
Вот кстати третья (gcc 4.4.0, Ubuntu 4.4.0-8ubuntu2):
gcc-4.4 -c -O -pipe -O -W -Wall -Wpointer-arith -Wno-unused-parameter
-Wunused-function -Wunused-variable -Wunused-value -Werror -g -O2 -I src/core
-I src/event -I src/event/modules -I src/os/unix -I objs -o objs/src/os/unix/ngx_channel.o src/os/unix/ngx_channel.c
cc1: warnings being treated as errors
src/os/unix/ngx_channel.c: In function ?ngx_write_channel?:
src/os/unix/ngx_channel.c:39: error: dereferencing type-punned pointer will
break strict-aliasing rules
src/os/unix/ngx_channel.c: In function ?ngx_read_channel?:
src/os/unix/ngx_channel.c:156: error: dereferencing type-punned pointer will
break strict-aliasing rules
Патч прилагается.
Вот тут IMHO memcpy вполне правилен, правильнее некуда.
Maxim Dounin
diff -r 72104cd120ec src/os/unix/ngx_channel.c
--- a/src/os/unix/ngx_channel.c Tue Dec 15 00:00:00 2009 +0300
+++ b/src/os/unix/ngx_channel.c Mon Dec 21 16:10:45 2009 +0300
@@ -36,7 +36,7 @@
cmsg.cm.cmsg_len = CMSG_LEN(sizeof(int));
cmsg.cm.cmsg_level = SOL_SOCKET;
cmsg.cm.cmsg_type = SCM_RIGHTS;
- *(int *) CMSG_DATA(&cmsg.cm) = ch->fd;
+ ngx_memcpy(CMSG_DATA(&cmsg.cm), &ch->fd, sizeof(ch->fd));
}
msg.msg_flags = 0;
@@ -153,7 +153,7 @@
return NGX_ERROR;
}
- ch->fd = *(int *) CMSG_DATA(&cmsg.cm);
+ ngx_memcpy(&ch->fd, CMSG_DATA(&cmsg.cm), sizeof(ch->fd));
}
if (msg.msg_flags & (MSG_TRUNC|MSG_CTRUNC)) {
_______________________________________________
nginx-ru mailing list
nginx-ru@xxxxxxxxx
http://nginx.org/mailman/listinfo/nginx-ru
|