* Re: [devel] Предлагаю приложить патчик к less
@ 2007-04-26 22:47 ` Dmitry V. Levin
2007-04-27 8:34 ` Slava Semushin
2007-04-27 8:38 ` Damir Shayhutdinov
0 siblings, 2 replies; 3+ messages in thread
From: Dmitry V. Levin @ 2007-04-26 22:47 UTC (permalink / raw)
To: ALT Devel discussion list
[-- 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 --]
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [devel] Предлагаю приложить патчик к less
2007-04-26 22:47 ` [devel] Предлагаю приложить патчик к less Dmitry V. Levin
@ 2007-04-27 8:34 ` Slava Semushin
2007-04-27 8:38 ` Damir Shayhutdinov
1 sibling, 0 replies; 3+ messages in thread
From: Slava Semushin @ 2007-04-27 8:34 UTC (permalink / raw)
To: ALT Devel discussion list
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
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [devel] Предлагаю приложить патчик к less
2007-04-26 22:47 ` [devel] Предлагаю приложить патчик к less Dmitry V. Levin
2007-04-27 8:34 ` Slava Semushin
@ 2007-04-27 8:38 ` Damir Shayhutdinov
1 sibling, 0 replies; 3+ messages in thread
From: Damir Shayhutdinov @ 2007-04-27 8:38 UTC (permalink / raw)
To: ALT Devel discussion list
> А какой смысл в этом патче, какую потенциальную проблему он призван исправить?
> Неужели есть на свете операционные системы, в которых close(-1) несёт
> угрозу?
Убрать лишний системный вызов aka context switch. ;)
Впрочем для интерактивных программ (таких как less) это не актуально -
пользователь тормозит больше чем любой context switch :)
А для всякой эмбедщины может быть полезно.
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2007-04-27 8:38 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2007-04-26 22:47 ` [devel] Предлагаю приложить патчик к less Dmitry V. Levin
2007-04-27 8:34 ` Slava Semushin
2007-04-27 8:38 ` Damir Shayhutdinov
ALT Linux Team development discussions
This inbox may be cloned and mirrored by anyone:
git clone --mirror http://lore.altlinux.org/devel/0 devel/git/0.git
# If you have public-inbox 1.1+ installed, you may
# initialize and index your mirror using the following commands:
public-inbox-init -V2 devel devel/ http://lore.altlinux.org/devel \
devel@altlinux.org devel@altlinux.ru devel@lists.altlinux.org devel@lists.altlinux.ru devel@linux.iplabs.ru mandrake-russian@linuxteam.iplabs.ru sisyphus@linuxteam.iplabs.ru
public-inbox-index devel
Example config snippet for mirrors.
Newsgroup available over NNTP:
nntp://lore.altlinux.org/org.altlinux.lists.devel
AGPL code for this site: git clone https://public-inbox.org/public-inbox.git