Date: Thu, 20 Jan 2011 00:54:12 +0000 (UTC) From: Rick Macklem <rmacklem@FreeBSD.org> To: cvs-src-old@freebsd.org Subject: cvs commit: src/sys/rpc clnt_dg.c clnt_vc.c svc_vc.c Message-ID: <201101200054.p0K0seRY066545@repoman.freebsd.org>
next in thread | raw e-mail | index | archive | help
rmacklem 2011-01-20 00:54:12 UTC
FreeBSD src repository
Modified files: (Branch: RELENG_8_2)
sys/rpc clnt_dg.c clnt_vc.c svc_vc.c
Log:
SVN rev 217617 on 2011-01-20 00:54:12Z 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.
Reviewed by: jhb (for head)
Approved by: re (kib)
Revision Changes Path
1.7.2.4.2.2 +3 -4 src/sys/rpc/clnt_dg.c
1.8.2.4.2.2 +5 -8 src/sys/rpc/clnt_vc.c
1.9.2.2.6.2 +2 -5 src/sys/rpc/svc_vc.c
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201101200054.p0K0seRY066545>