Илья Шипицин
October 20, 2017 10:44AM
и вот, кажется, интересные штуки

# grep '(error' typescript
[src/go/unit/nxt_go_port_memory.c:52]: (error) Undefined behavior: Variable
'name' is used as parameter and destination in s[n]printf().
[src/nxt_lib.c:96]: (error) Uninitialized variable: n
[src/nxt_main_process.c:398]: (error) Memory pointed to by 'start' is freed
twice.
[src/nxt_php_sapi.c:313]: (error) Uninitialized struct member:
script_name.start
[src/nxt_port_memory.c:475]: (error) Dereferencing 'mmap_handler' after it
is deallocated / released


20 октября 2017 г., 19:30 пользователь Илья Шипицин <chipitsine@gmail.com>
написал:

> Валентин, посмотрите вот эти штуки ?
>
> # grep '(warning' typescript
> [src/nxt_file_cache.c:290] -> [src/nxt_file_cache.c:302]: (warning) Either
> the condition 'handler==NULL' is redundant or there is possible null
> pointer dereference: handler.
> [src/nxt_port_socket.c:498] -> [src/nxt_port_socket.c:505]: (warning)
> Either the condition 'b==NULL' is redundant or there is possible null
> pointer dereference: b.
>
>
> и, кажется, не работают тесты при "make tests" (вот это прямо
> напрашивается в travis)
>
> я хотел этим заняться, завал, к сожалению еще на пару месяцев
>
> 20 октября 2017 г., 19:25 пользователь Валентин Бартенев <vbart@nginx.com>
> написал:
>
> On Friday 20 October 2017 16:48:31 Slawa Olhovchenkov wrote:
>> > On Fri, Oct 20, 2017 at 04:42:54PM +0300, Maksim Kulik wrote:
>> >
>> > > Так в таком случае использование unit еще выгоднее: ему не надо
>> держать
>> > > master-процесс для каждой версии php, не говоря о процессе для каждого
>> > > пользователя.
>> > >
>> > > P.S. Может я немного отстал от актуальных знаний о PHP-FPM, но зачем
>> под
>> > > каждого пользователя запускать отдельный master-процесс? Достаточно
>> ведь
>> > > завести для конкретного пользователя свой pool (работающий от имени
>> этого
>> > > пользователя), а мастер-процесс будет всегда один. Если я ошибаюсь -
>> > > скиньте, плиз, линку на почту где можно подробнее почитать об
>> опасности
>> > > запуска одного мастер-процесса для разных пользователей.
>> >
>> > Это достаточно самоочевидно для любого, кто немного интересуется
>> > безопасностью.
>> > Ну и для програмистов эдак начиная примерно с 15+ лет опыта. Но лучше
>> 20+.
>> > В общем когда приходит понимание, что программ без ошибок не бывает.
>> > Тогда доходит и мысль о том, что общий мастер-процесс на всех должен
>> > иметь возможность читать конфиг принадлежащий любому пользователю и
>> > перезапускать пул(ы) от любого пользовательского UID, а это уже есть
>> > некторая дыра, т.к. он получается должен быть рутовым.
>> > В модели отдельного мастера на пользователя он после запуска
>> > безвозвратно дропает свои привелегии и эта схема в целом меньше
>> > подвержена протечкам.
>> [..]
>>
>> Основной процесс Unit-а, который работает от рута, не читает конфигов
>> и не взаимодействует с пользователями и их приложениями.
>>
>> Каждое приложение со своей конфигурацией полностью изолировано. Точно
>> также,
>> как были бы изолированы отдельные процессы php-fpm, запущенные независимо
>> друг
>> от друга на одной машине.
>>
>> --
>> Валентин Бартенев
>> _______________________________________________
>> nginx-ru mailing list
>> nginx-ru@nginx.org
>> http://mailman.nginx.org/mailman/listinfo/nginx-ru
>>
>
>
_______________________________________________
nginx-ru mailing list
nginx-ru@nginx.org
http://mailman.nginx.org/mailman/listinfo/nginx-ru
Subject Author Posted

unit-0.2 beta release

Igor Sysoev October 20, 2017 03:44AM

Re: unit-0.2 beta release

Anton Kiryushkin October 20, 2017 04:28AM

Re: unit-0.2 beta release

Andrey Velikoredchanin October 20, 2017 04:46AM

Re: unit-0.2 beta release

Илья Шипицин October 20, 2017 09:02AM

Re: unit-0.2 beta release

Maksim Kulik October 20, 2017 09:30AM

Re: unit-0.2 beta release

Виктор Вислобоков October 20, 2017 09:38AM

Re: unit-0.2 beta release

Maksim Kulik October 20, 2017 09:44AM

Re: unit-0.2 beta release

Slawa Olhovchenkov October 20, 2017 09:50AM

Re: unit-0.2 beta release

Никита Козлов October 20, 2017 09:54AM

Re: unit-0.2 beta release

Maksim Kulik October 20, 2017 10:00AM

Re: unit-0.2 beta release

Slawa Olhovchenkov October 20, 2017 10:08AM

Re: unit-0.2 beta release

Валентин Бартенев October 20, 2017 10:26AM

Re: unit-0.2 beta release

Виктор Вислобоков October 20, 2017 10:28AM

Re: unit-0.2 beta release

Валентин Бартенев October 20, 2017 11:06AM

Re: unit-0.2 beta release

Виктор Вислобоков October 20, 2017 11:22AM

Re: unit-0.2 beta release

Igor Sysoev October 20, 2017 11:28AM

Re: unit-0.2 beta release

Виктор Вислобоков October 20, 2017 11:30AM

Re: unit-0.2 beta release

Валентин Бартенев October 20, 2017 11:56AM

Re: unit-0.2 beta release

Виктор Вислобоков October 20, 2017 12:22PM

Re: unit-0.2 beta release

Andrey Velikoredchanin October 20, 2017 01:34PM

Re: unit-0.2 beta release

Pavel V. October 20, 2017 01:40PM

Re: unit-0.2 beta release

Andrey Oktyabrskiy October 20, 2017 03:52PM

Re: unit-0.2 beta release

Igor Sysoev October 20, 2017 03:58PM

Re: unit-0.2 beta release

Andrey Oktyabrskiy October 20, 2017 04:20PM

Re: unit-0.2 beta release

Иван December 15, 2017 10:00AM

Re: unit-0.2 beta release

Валентин Бартенев December 15, 2017 12:40PM

Re: unit-0.2 beta release

Иван December 15, 2017 01:56PM

Re: unit-0.2 beta release

Валентин Бартенев December 16, 2017 08:06AM

Re: unit-0.2 beta release

Валентин Бартенев December 29, 2017 01:14PM

Re: unit-0.2 beta release

Slawa Olhovchenkov October 20, 2017 03:40PM

Re: unit-0.2 beta release

Виктор Вислобоков October 20, 2017 03:42PM

Re: unit-0.2 beta release

Slawa Olhovchenkov October 20, 2017 04:28PM

Re: unit-0.2 beta release

Vadim A. Misbakh-Soloviov October 22, 2017 03:20AM

Re: unit-0.2 beta release

Andrey Velikoredchanin October 24, 2017 04:50AM

Re: unit-0.2 beta release

NickShadrin October 24, 2017 10:54AM

Re: unit-0.2 beta release

NickShadrin October 24, 2017 11:42AM

Re: unit-0.2 beta release

Валентин Бартенев October 24, 2017 12:10PM

Re: unit-0.2 beta release

Илья Шипицин October 20, 2017 10:32AM

Re: unit-0.2 beta release

Илья Шипицин October 20, 2017 10:44AM

Re: unit-0.2 beta release

Pavel V. October 20, 2017 10:12AM

Re: unit-0.2 beta release

Виктор Вислобоков October 20, 2017 10:14AM

Re: unit-0.2 beta release

Slawa Olhovchenkov October 20, 2017 10:22AM

Re: unit-0.2 beta release

Виктор Вислобоков October 20, 2017 10:26AM

Re: unit-0.2 beta release

Igor Sysoev October 20, 2017 10:30AM

Re: unit-0.2 beta release

Виктор Вислобоков October 20, 2017 10:36AM

Re: unit-0.2 beta release

Валентин Бартенев October 20, 2017 10:44AM

Re: unit-0.2 beta release

Igor Sysoev October 20, 2017 10:46AM

Re: unit-0.2 beta release

Slawa Olhovchenkov October 20, 2017 10:46AM

Re: unit-0.2 beta release

Алексей Сундуков October 26, 2017 02:44AM

Re: unit-0.2 beta release

Peter B. Pokryshev October 20, 2017 09:48AM

Re: unit-0.2 beta release

Andrey Velikoredchanin October 20, 2017 10:04AM

Re: unit-0.2 beta release

Валентин Бартенев October 20, 2017 10:32AM

Re: unit-0.2 beta release

S.A.N October 20, 2017 03:03PM

Re: unit-0.2 beta release

Igor Sysoev October 20, 2017 03:28PM



Sorry, only registered users may post in this forum.

Click here to login

Online Users

Guests: 233
Record Number of Users: 8 on April 13, 2023
Record Number of Guests: 421 on December 02, 2018
Powered by nginx      Powered by FreeBSD      PHP Powered      Powered by MariaDB      ipv6 ready