From owner-freebsd-current  Wed Nov 22 20:33:38 2000
Delivered-To: freebsd-current@freebsd.org
Received: from clmboh1-smtp1.columbus.rr.com (clmboh1-smtp1.columbus.rr.com [65.24.0.110])
	by hub.freebsd.org (Postfix) with ESMTP
	id 9790437B479; Wed, 22 Nov 2000 20:33:34 -0800 (PST)
Received: from columbus.rr.com (dzerkel@dhcp065-024-166-103.columbus.rr.com [65.24.166.103])
	by clmboh1-smtp1.columbus.rr.com (8.9.3/8.9.3) with ESMTP id XAA29788;
	Wed, 22 Nov 2000 23:30:14 -0500 (EST)
Message-ID: <3A1C9E1D.574887CE@columbus.rr.com>
Date: Wed, 22 Nov 2000 23:33:33 -0500
From: "Danny J. Zerkel" <dzerkel@columbus.rr.com>
Organization: Zerkular Software
X-Mailer: Mozilla 4.76 [en] (X11; U; Linux 2.2.12 i386)
X-Accept-Language: Russian, ru, en
MIME-Version: 1.0
To: FreeBSD Current <freebsd-current@FreeBSD.ORG>, dillon@FreeBSD.ORG
Subject: RE: COMPAT_SVR4 broken after uipc_syscalls commit (1.77)
Content-Type: text/plain; charset=us-ascii
Content-Transfer-Encoding: 7bit
Sender: owner-freebsd-current@FreeBSD.ORG
Precedence: bulk
X-Loop: FreeBSD.ORG

Okay, this time I'll even include the entire patch...

-- Danny J. Zerkel
dzerkel@columbus.rr.com

--- svr4_stream.c.orig  Thu Aug 31 18:54:05 2000
+++ svr4_stream.c       Wed Nov 22 22:39:00 2000
@@ -162,7 +162,7 @@
        struct uio ktruio;
 #endif
 
-       error = getsock(p->p_fd, s, &fp);
+       error = holdsock(p->p_fd, s, &fp);
        if (error)
                return (error);
        auio.uio_iov = mp->msg_iov;
@@ -174,13 +174,17 @@
        auio.uio_resid = 0;
        iov = mp->msg_iov;
        for (i = 0; i < mp->msg_iovlen; i++, iov++) {
-               if ((auio.uio_resid += iov->iov_len) < 0)
+               if ((auio.uio_resid += iov->iov_len) < 0) {
+                       fdrop(fp, p);
                        return (EINVAL);
+               }
        }
        if (mp->msg_name) {
                error = getsockaddr(&to, mp->msg_name, mp->msg_namelen);
-               if (error)
+               if (error) {
+                       fdrop(fp, p);
                        return (error);
+               }
        } else
                to = 0;
        if (mp->msg_control) {
@@ -229,6 +233,7 @@
 bad:
        if (to)
                FREE(to, M_SONAME);
+       fdrop(fp, p);
        return (error);
 }
 
@@ -253,7 +258,7 @@
        struct uio ktruio;
 #endif
 
-       error = getsock(p->p_fd, s, &fp);
+       error = holdsock(p->p_fd, s, &fp);
        if (error)
                return (error);
        auio.uio_iov = mp->msg_iov;
@@ -265,8 +270,10 @@
        auio.uio_resid = 0;
        iov = mp->msg_iov;
        for (i = 0; i < mp->msg_iovlen; i++, iov++) {
-               if ((auio.uio_resid += iov->iov_len) < 0)
+               if ((auio.uio_resid += iov->iov_len) < 0) {
+                       fdrop(fp, p);
                        return (EINVAL);
+               }
        }
 #ifdef KTRACE
        if (KTRPOINT(p, KTR_GENIO)) {
@@ -352,6 +359,7 @@
                FREE(fromsa, M_SONAME);
        if (control)
                m_freem(control);
+       fdrop(fp, p);
        return (error);
 }


To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe freebsd-current" in the body of the message