From owner-cvs-all Thu Oct 25 9:20:39 2001 Delivered-To: cvs-all@freebsd.org Received: from net2.dinoex.sub.org (net2.dinoex.de [212.184.201.182]) by hub.freebsd.org (Postfix) with ESMTP id 90B5737B401; Thu, 25 Oct 2001 09:20:31 -0700 (PDT) Received: from gate.dinoex.sub.org (dinoex@localhost) by net2.dinoex.sub.org (8.11.6/8.11.6) with BSMTP id f9PGK6e10405; Thu, 25 Oct 2001 18:20:06 +0200 (CEST) (envelope-from dirk.meyer@dinoex.sub.org) To: cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org, nectar@nectar.com Message-ID: From: dirk.meyer@dinoex.sub.org (Dirk Meyer) Organization: privat Subject: Re: cvs commit: ports/security/openssh Makefile ports/security/openssh/files patch-coredump ports/security/openssh-portable Makefile ports/security/openssh-portable/files patch-coredump Date: Thu, 25 Oct 2001 18:14:15 +0200 X-Mailer: Dinoex 1.77 References: <200110240716.f9O7GnQ17535@freefall.freebsd.org> <200110240716.f9O7GnQ17535@freefall.freebsd.org> <20011025070926.A73086@madman.nectar.com> X-Gateway: ZCONNECT gate.dinoex.sub.org [UNIX/Connect 0.93] X-PGP-Fingerprint: 44 16 EC 0A D3 3A 4F 28 8A 8A 47 93 F1 CF 2F 12 X-Copyright: (C) Copyright 2001 by Dirk Meyer -- All rights reserved. X-PGP-Key-Avail: mailto:pgp-public-keys@keys.de.pgp.net Subject:GET 0x331CDA5D X-ZC-VIA: 20011025000000S+2@dinoex.sub.org X-Accept-Language: de,en X-Noad: Please don't send me ad's by mail. I'm bored by this type of mail. X-Note: sending SPAM is a violation of both german and US law and will at least trigger a complaint at your provider's postmaster. X-No-Archive: yes Sender: owner-cvs-all@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG Jacques A. Vidrine wrote: > Please do not use more than one patch file per source file. Now both > `patch-session.c' and `patch-coredump' modify `session.c'. > > Also, do you have more information on the problem that is supposedly > solved by this patch? The patch simply initializes `env' to NULL. > However, `env' is not used before being initialized that I can see. > Did I miss something? http://www.FreeBSD.org/cgi/search.cgi?words=ht3BBA7E1E.931F087D%40prism.gatech.edu&max=25&sort=date&index=recent&source=freebsd-stable The patch Ryan suggested was confirmed in the base system. But I failed to see that it can't fail on opennssh 2.9.9, where the handling was totally rewritten inside. kind regards Dirk - Dirk Meyer, Im Grund 4, 34317 Habichtswald, Germany ryanb@goddamnbastard.org (ryan beasley) wrote: > Sorry for replying to myself, but I drafted this little "patch" that > should take care of the segfault in session.c. After thinking about > this for a bit, it probably should've been _really_ obvious to me > after the first lookover ... > > --- src/crypto/openssh/session.c.orig Wed Oct 24 00:35:57 2001 > +++ src/crypto/openssh/session.c Wed Oct 24 00:36:26 2001 > @@ -1003,7 +1003,7 @@ > char cmd[1024]; > FILE *f =3D NULL; > u_int envsize, i; > - char **env; > + char **env =3D NULL; > extern char **environ; > struct stat st; > char *argv[10]; > To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message