ALT Linux Team development discussions
 help / color / mirror / Atom feed
From: Eugene Prokopiev <enp@altlinux.org>
To: ALT Linux Team development discussions <devel@lists.altlinux.org>
Subject: Re: [devel] I: gcc4.4-4.4.0-alt4
Date: Wed, 15 Jul 2009 08:57:06 +0400
Message-ID: <f7a739430907142157j44bb2d61w7c424ab45f59bc9b@mail.gmail.com> (raw)
In-Reply-To: <20090714172241.GK5822@wrars-comp.wrarsdomain>

14 июля 2009 г. 21:22 пользователь Andrey Rahmatullin
(wrar@altlinux.ru) написал:
> On Tue, Jul 14, 2009 at 11:34:55AM +0400, Eugene Prokopiev wrote:
>> с благодарностью приму патч для callweaver :)
> channels/chan_sip.c
> В 1922 в поле struct sip_pvt::via пишется snprintf'ом строка максимальной
> длины SIP_MAX_LINE_LEN. При этом struct sip_pvt::via в 811 объявлен
> размером 256, а SIP_MAX_LINE_LEN в 572 равен 1024. Я не знаю, что с этим
> делать, пишите в апстрим. Допустимые для подобного кода решения (не
> вдаваясь в то, что код делает) могут быть расширение поля (это ломает
> ABI), уменьшение используемой константы (может быть некорректно, если эта
> константа используется при записи в более длинные буферы) или
> использование другой константы.
>
> res/res_sqlite.c
> В 417 в буфер sqlbuf длиной 1024 пишутся strncat'ом строки длиной 1024+1
> (sic! завершающий \0 в третьем параметре strncat не учитывается, потому
> там должно быть 1023).
> В 418 туда же дописывается ещё 1024+1 байта, и это вообще неправильно, т.к.
> третий параметр указывает, сколько писать ещё, а не сколько должно
> получиться. Надо писать 1024-strlen(sqlbuf) плюс/минус 1, ну или strlcat
> использовать (непереносимо).

Спасибо, пойду в апстрим

-- 
С уважением,
Прокопьев Евгений

  reply	other threads:[~2009-07-15  4:57 UTC|newest]

Thread overview: 32+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-07-13  8:47 Dmitry V. Levin
2009-07-13 13:07 ` Dmitry V. Levin
2009-07-13 13:29   ` Alexey Tourbin
2009-07-13 13:43     ` Dmitry V. Levin
2009-07-13 14:37   ` Andrey Rahmatullin
2009-07-14  3:23   ` Alexander Bokovoy
2009-07-14 10:56     ` Dmitry V. Levin
2009-07-14 11:50       ` Alexander Bokovoy
2009-07-14 11:57         ` Dmitry V. Levin
2009-07-14 21:18     ` Vitaly Lipatov
2009-07-15  4:16       ` Alexander Bokovoy
2009-07-15 21:59         ` Vitaly Lipatov
2009-07-16  3:13           ` Alexander Bokovoy
2009-07-16  7:19             ` Evgeny Sinelnikov
2009-07-16  9:01               ` Alexander Bokovoy
2009-08-08 10:04     ` Alexander Bokovoy
2009-07-14  4:09   ` REAL
2009-07-14  7:14   ` Andrey Rahmatullin
2009-07-14  7:34     ` Eugene Prokopiev
2009-07-14  7:53       ` REAL
2009-07-14 13:16         ` Dmitry V. Levin
2009-07-14 22:28           ` [devel] I: gcc4.4-4.4.0-alt6 Dmitry V. Levin
2009-07-14 17:01         ` [devel] I: gcc4.4-4.4.0-alt4 Andrey Rahmatullin
2009-07-14  8:52       ` Andrey Rahmatullin
2009-07-14 17:22       ` Andrey Rahmatullin
2009-07-15  4:57         ` Eugene Prokopiev [this message]
2009-07-14 22:31     ` Dmitry V. Levin
2009-07-14  8:20   ` Денис Смирнов
2009-07-14 21:41     ` Vitaly Lipatov
2009-07-14 22:44     ` Ilya Mashkin
2009-07-14 23:00       ` Dmitry V. Levin
2009-07-14 17:27   ` Andrey Rahmatullin

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=f7a739430907142157j44bb2d61w7c424ab45f59bc9b@mail.gmail.com \
    --to=enp@altlinux.org \
    --cc=devel@lists.altlinux.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

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