From owner-cvs-src-old@FreeBSD.ORG Tue Jan 18 01:20:33 2011 Return-Path: Delivered-To: cvs-src-old@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 87D3310657C3 for ; Tue, 18 Jan 2011 01:20:33 +0000 (UTC) (envelope-from rmacklem@FreeBSD.org) Received: from repoman.freebsd.org (repoman.freebsd.org [IPv6:2001:4f8:fff6::29]) by mx1.freebsd.org (Postfix) with ESMTP id 758AD8FC18 for ; Tue, 18 Jan 2011 01:20:33 +0000 (UTC) Received: from repoman.freebsd.org (localhost [127.0.0.1]) by repoman.freebsd.org (8.14.4/8.14.4) with ESMTP id p0I1KX67074036 for ; Tue, 18 Jan 2011 01:20:33 GMT (envelope-from rmacklem@repoman.freebsd.org) Received: (from svn2cvs@localhost) by repoman.freebsd.org (8.14.4/8.14.4/Submit) id p0I1KXww074035 for cvs-src-old@freebsd.org; Tue, 18 Jan 2011 01:20:33 GMT (envelope-from rmacklem@repoman.freebsd.org) Message-Id: <201101180120.p0I1KXww074035@repoman.freebsd.org> X-Authentication-Warning: repoman.freebsd.org: svn2cvs set sender to rmacklem@repoman.freebsd.org using -f From: Rick Macklem Date: Tue, 18 Jan 2011 01:20:15 +0000 (UTC) To: cvs-src-old@freebsd.org X-FreeBSD-CVS-Branch: RELENG_8 Subject: cvs commit: src/sys/rpc clnt_dg.c clnt_vc.c svc_vc.c X-BeenThere: cvs-src-old@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: **OBSOLETE** CVS commit messages for the src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 18 Jan 2011 01:20:33 -0000 rmacklem 2011-01-18 01:20:15 UTC FreeBSD src repository Modified files: (Branch: RELENG_8) sys/rpc clnt_dg.c clnt_vc.c svc_vc.c Log: SVN rev 217527 on 2011-01-18 01:20:15Z by rmacklem MFC: r217242 Fix a bug in the client side krpc where it was, sometimes erroneously, assumed that 4 bytes of data were in the first mbuf of a list by replacing the bcopy() with m_copydata(). Also, replace the uses of m_pullup(), which can fail for reasons other than not enough data, with m_copydata(). For the cases where it isn't known that there is enough data in the mbuf list, check first via m_len and m_length(). This is believed to fix a problem reported by dpd at dpdtech.com and george+freebsd at m5p.com. Revision Changes Path 1.7.2.5 +3 -4 src/sys/rpc/clnt_dg.c 1.8.2.5 +5 -8 src/sys/rpc/clnt_vc.c 1.9.2.3 +2 -5 src/sys/rpc/svc_vc.c