Re: [devel] Предлагаю приложить патчик к less

Re: [devel] Предлагаю приложить патчик к less

[Dmitry V. Levin]

[-- Attachment #1: Type: text/plain, Size: 613 bytes --]

Hi,

On Thu, Apr 26, 2007 at 11:37:40AM +0400, Slava Semushin wrote:
> В NetBSD к less прикладывают патчик, который, насколько понимаю,
> предотвращает закрытие файла, который не был открыт. Это было найдено
> с помощью Coverity. Патч не большой, доступен здесь:
> 
> http://cvsweb.netbsd.org/bsdweb.cgi/pkgsrc/misc/less/patches/patch-ac?rev=1.1&content-type=text/x-cvsweb-markup
> 
> Я предлагаю его приложить к нашей сборке.

А какой смысл в этом патче, какую потенциальную проблему он призван исправить?
Неужели есть на свете операционные системы, в которых close(-1) несёт
угрозу?


-- 
ldv

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: [devel] Предлагаю приложить патчик к less

[Slava Semushin]

27.04.07, Dmitry V. Levin<ldv / altlinux.org> написал(а):
[...]
> А какой смысл в этом патче, какую потенциальную проблему он призван исправить?

Смысл в том, что если файл не открылся, то и закрывать его не надо
пытаться. Проблемы для пользователя, здесь нет, зато есть не очень
хороший код.

> Неужели есть на свете операционные системы, в которых close(-1) несёт
> угрозу?

Вряд ли есть :) Просто close() вернёт ошибку да и всё.


Ок, понял, что патч не нужен...

-- 
+ Slava Semushin | slava.semushin @ gmail.com
+ ALT Linux Team | php-coder @ altlinux.ru

Re: [devel] Предлагаю приложить патчик к less

[Damir Shayhutdinov]

> А какой смысл в этом патче, какую потенциальную проблему он призван исправить?
> Неужели есть на свете операционные системы, в которых close(-1) несёт
> угрозу?
Убрать лишний системный вызов aka context switch. ;)

Впрочем для интерактивных программ (таких как less) это не актуально -
пользователь тормозит больше чем любой context switch :)

А для всякой эмбедщины может быть полезно.