On Mon, Dec 21, 2009 at 04:16:16PM +0300, Maxim Dounin wrote:

> 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 вполне правилен, правильнее некуда.

Только, если gcc его заинланит. А то очень похоже на

len = strlen("статическая строка");

> 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@nginx.org
> http://nginx.org/mailman/listinfo/nginx-ru


--
Игорь Сысоев
http://sysoev.ru

_______________________________________________
nginx-ru mailing list
nginx-ru@nginx.org
http://nginx.org/mailman/listinfo/nginx-ru