Да, нужно доработать патч. Согласен.
С уважением,
Владимир Коробов
-Original Message-
From: Slawa Olhovchenkov
Sent: Wednesday, October 5, 2022 1:10 PM
To: Korobov Vladimir via nginx-ru
Subject: Re: Исправления срабатываний статического анализатора.
On Wed, Oct 05, 2022 at 09:24
Да, патч нужно дорабатывать. Согласен.
С уважением,
Владимир Коробов
-Original Message-
From: Evgeniy Berdnikov
Sent: Wednesday, October 5, 2022 12:33 PM
To: nginx-ru@nginx.org
Subject: Re: Исправления срабатываний статического анализатора.
On Wed, Oct 05, 2022 at 09:24:19AM +
; Владимир Коробов
>
>
> -Original Message-
> From: Slawa Olhovchenkov
> Sent: Wednesday, October 5, 2022 12:13 PM
> To: Korobov Vladimir via nginx-ru
> Subject: Re: Исправления срабатываний статического анализатора.
>
> On Wed, Oct 05, 2022 at 05:22:10
On Wed, Oct 05, 2022 at 09:24:19AM +, Korobov Vladimir via nginx-ru wrote:
> Окей, неправально выразился.
> Я понимаю, что делает этот патч и он не делает хуже.
Я написал, почему этот патч делает хуже, причём СИЛЬНО хуже чем было.
И Максим Дунин высказался. Вы можете что-то возразить
Окей, неправально выразился.
Я понимаю, что делает этот патч и он не делает хуже.
С уважением,
Владимир Коробов
-Original Message-
From: Slawa Olhovchenkov
Sent: Wednesday, October 5, 2022 12:13 PM
To: Korobov Vladimir via nginx-ru
Subject: Re: Исправления срабатываний статического
t;кажется"?
ны написал какой-то код, но сам не понимаешь что он делает?
а не делает ли он хуже?
> С уважением,
> Владимир Коробов
>
>
> -Original Message-
> From: Slawa Olhovchenkov
> Sent: Tuesday, October 4, 2022 4:05 PM
> To: Korobov Vladimir via nginx-
Проверять на NULL, конечно, надо. Тем более во всех файлах, в которых я внёс
изменения, такие проверки есть. И выглядит так, что в указанных местах такие
проверки добавить забыли.
На счёт "тупого выбрасывания кусков кода" согласен, добавлю логирование.
С уважением,
Владимир Коробов
To: Korobov Vladimir via nginx-ru
Subject: Re: Исправления срабатываний статического анализатора.
On Tue, Oct 04, 2022 at 12:00:57PM +, Korobov Vladimir via nginx-ru wrote:
> Добрый день
>
> После проверки исходного кода статическим анализатором (Svace
> https://www.ispras.ru/
05.10.2022 2:54, Evgeniy Berdnikov пишет:
> On Wed, Oct 05, 2022 at 02:34:33AM +0700, Eugene Grosbein wrote:
>> 05.10.2022 0:53, Evgeniy Berdnikov пишет:
>>> Был бы я пользователем, я бы тоже так считал, наверное... Но поскольку я
>>> сисадмин с некоторым запилом в разработку, то думаю иначе:
On Wed, Oct 05, 2022 at 02:34:33AM +0700, Eugene Grosbein wrote:
> 05.10.2022 0:53, Evgeniy Berdnikov пишет:
> > Был бы я пользователем, я бы тоже так считал, наверное... Но поскольку я
> > сисадмин с некоторым запилом в разработку, то думаю иначе: вставить в свою
> > софтину полноценный
05.10.2022 0:53, Evgeniy Berdnikov пишет:
>> Другой вопрос, что потом делать, если вдруг: молча восстановиться и ехать
>> дальше,
>> или не молча, а с сообщением в лог, или выдать даже stack trace и выйти. Но
>> что угодно лучше сырого сегфолта.
>
> Был бы я пользователем, я бы тоже так
Hello!
On Tue, Oct 04, 2022 at 11:33:14PM +0700, Eugene Grosbein wrote:
> 04.10.2022 20:11, Evgeniy Berdnikov пишет:
> > On Tue, Oct 04, 2022 at 12:00:57PM +, Korobov Vladimir via nginx-ru
> > wrote:
> >>После проверки исходного кода статическим анализатором (Svace
> >>
On Tue, Oct 04, 2022 at 11:33:14PM +0700, Eugene Grosbein wrote:
> 04.10.2022 20:11, Evgeniy Berdnikov пишет:
> > При потенциальной возможности зануления указателя следует ловить и
> > обрабатывать такое исключение. В противном случае нет смысла в проверке.
> > Задача же не в ублажении тупых
04.10.2022 20:11, Evgeniy Berdnikov пишет:
> On Tue, Oct 04, 2022 at 12:00:57PM +, Korobov Vladimir via nginx-ru wrote:
>>После проверки исходного кода статическим анализатором (Svace
>>https://www.ispras.ru/technologies/svace/) выделено несколько потенциально
>>опасных мест,
On Tue, Oct 04, 2022 at 12:00:57PM +, Korobov Vladimir via nginx-ru wrote:
>После проверки исходного кода статическим анализатором (Svace
>https://www.ispras.ru/technologies/svace/) выделено несколько потенциально
>опасных мест, закрывающихся приложенным патчем.
Тупое
On Tue, Oct 04, 2022 at 12:00:57PM +, Korobov Vladimir via nginx-ru wrote:
> Добрый день
>
> После проверки исходного кода статическим анализатором (Svace
> https://www.ispras.ru/technologies/svace/) выделено несколько потенциально
> опасных мест, закрывающихся приложенным патчем.
>
>
Добрый день
После проверки исходного кода статическим анализатором (Svace
https://www.ispras.ru/technologies/svace/) выделено несколько потенциально
опасных мест, закрывающихся приложенным патчем.
Прошу рассмотреть возможность включения указанных изменений в исходный код
проекта.
С
17 matches
Mail list logo