ALT Linux Sisyphus discussions
 help / color / mirror / Atom feed
From: "Dmitry V. Levin" <ldv@altlinux.org>
To: ALT Linux Sisyphus discussions <sisyphus@lists.altlinux.org>
Subject: Re: [sisyphus] Q: /usr/bin/openssl segmentation fault
Date: Wed, 10 Dec 2008 14:51:46 +0300
Message-ID: <20081210115146.GS21988@wo.int.altlinux.org> (raw)
In-Reply-To: <921f6bb40812092121s357c9adeof21e9a7bf2a90f8f@mail.gmail.com>

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

On Wed, Dec 10, 2008 at 08:21:43AM +0300, Evgeny Sinelnikov wrote:
> 10 декабря 2008 г. 1:48 пользователь Dmitry Lebkov <dima@sakhalin.ru> написал:
> > Dmitry V. Levin пишет:
> >> On Wed, Dec 10, 2008 at 03:15:08AM +1000, Dmitry Lebkov wrote:
> >> [...]
> >>> Таки я нашел упоминание подобной проблемы и соответствующее "лекарство":
> >>> http://marc.info/?l=openssl-dev&m=121993344410613&w=2
> >>
> >> Странно, по идее,
> >> PKCS12_add_friendlyname(bag, name, -1)
> >> и
> >> PKCS12_add_friendlyname(bag, name, strlen(name))
> >> это должно быть одно и то же.
> >
> > Но таки оно валится по SIGSEGV в первом случае и нормально отрабатывает
> > во втором -- проверено на локально пересобранном пакете.
> 
> Да, но всё же странный патч... Забавно и то, что опубликован он в
> конце августа, а в сентябрьский релиз так и не попал... То есть, не
> мне одному он мог показаться странным...

Я хотел этим сказать, что если в одном месте кода пришлось заменить -1 на
strlen, то почему в других местах не надо этого делать?  И наоборот.

На этот вопрос нет ответа без оглядки на реализацию
PKCS12_add_friendlyname.


-- 
ldv

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

  reply	other threads:[~2008-12-10 11:51 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-12-09 17:03 Dmitry Lebkov
2008-12-09 17:15 ` Dmitry Lebkov
2008-12-09 17:40   ` Led
2008-12-09 18:06     ` Dmitry V. Levin
2008-12-09 18:21       ` Led
2008-12-09 18:18   ` Dmitry V. Levin
2008-12-09 22:48     ` Dmitry Lebkov
2008-12-09 22:56       ` Dmitry V. Levin
2008-12-09 23:20         ` Dmitry Lebkov
2008-12-10  5:21       ` Evgeny Sinelnikov
2008-12-10 11:51         ` Dmitry V. Levin [this message]
2008-12-10 12:00           ` Konstantin A. Lepikhov

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=20081210115146.GS21988@wo.int.altlinux.org \
    --to=ldv@altlinux.org \
    --cc=sisyphus@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 Sisyphus discussions

This inbox may be cloned and mirrored by anyone:

	git clone --mirror http://lore.altlinux.org/sisyphus/0 sisyphus/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 sisyphus sisyphus/ http://lore.altlinux.org/sisyphus \
		sisyphus@altlinux.ru sisyphus@altlinux.org sisyphus@lists.altlinux.org sisyphus@lists.altlinux.ru sisyphus@lists.altlinux.com sisyphus@linuxteam.iplabs.ru sisyphus@list.linux-os.ru
	public-inbox-index sisyphus

Example config snippet for mirrors.
Newsgroup available over NNTP:
	nntp://lore.altlinux.org/org.altlinux.lists.sisyphus


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git