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
> использовать (непереносимо).
Спасибо, пойду в апстрим
--
С уважением,
Прокопьев Евгений
next prev parent 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