From owner-freebsd-security Mon Apr 22 14:16:44 2002 Delivered-To: freebsd-security@freebsd.org Received: from atlantis.dp.ua (atlantis.dp.ua [193.108.46.1]) by hub.freebsd.org (Postfix) with ESMTP id 9554137BABF for ; Mon, 22 Apr 2002 14:15:34 -0700 (PDT) Received: from localhost (dmitry@localhost) by atlantis.dp.ua (8.11.1/8.11.1) with ESMTP id g3MKlIX65284 for ; Mon, 22 Apr 2002 23:47:18 +0300 (EEST) (envelope-from dmitry@atlantis.dp.ua) Date: Mon, 22 Apr 2002 23:47:18 +0300 (EEST) From: Dmitry Pryanishnikov To: freebsd-security@freebsd.org Subject: Re: FreeBSD Security Advisory FreeBSD-SA-02:23.stdio Message-ID: <20020422234500.F64803-100000@atlantis.dp.ua> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: owner-freebsd-security@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.org Hello! Has this patch (stdio.patch) been verified properly? It seems that it contains obvious error: + int fd, i, error, flags, devnull; ... + flags = FREAD | FWRITE; + error = vn_open(&nd, &flags, 0); Note that second argument of vn_open must be of type int, not int * (so error = vn_open(&nd, flags, 0) should be correct). This gives the following warning during compilation: cc -c -O -pipe -Wall -Wredundant-decls -Wnested-externs -Wstrict-prototypes -Wmissing-prototypes -Wpointer-arith -Winline -Wcast-qual -fformat-extensions -ansi -nostdinc -I- -I. -I../.. -I../../../include -I../../contrib/ipfilter -D_KERNEL -include opt_global.h -elf -fno-builtin -mpreferred-stack-boundary=2 ../../kern/kern_descrip.c ../../kern/kern_descrip.c: In function `fdcheckstd': ../../kern/kern_descrip.c:1216: warning: passing arg 2 of `vn_open' makes integer from pointer without a cast Please check this code! Sincerely, Dmitry Atlantis ISP, System Administrator e-mail: dmitry@atlantis.dp.ua nic-hdl: LYNX-RIPE To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-security" in the body of the message