d/freebsd-skq
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix a bug in the userland rpc library, where it would use a

Browse Source 
negative return value from write to update its position in
a buffer. The patch, courtesy of Andrey Simonenko, also simplifies
a conditional by removing the "i != cnt" clause, since it is
always true at this point in the code. The bug caused problems
for mountd, when it generated a large reply to an exports RPC
request.

Submitted by:	simon at comsys.ntu-kpi.kiev.ua
MFC after:	2 weeks
This commit is contained in:
Rick Macklem 2011-04-10 14:11:07 +00:00
parent 9d12ecf01e
commit e2c83f74f4
1 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
lib/libc/rpc/svc_vc.c
View File

@ -546,7 +546,7 @@ write_vc(xprtp, buf, len)
cd->strm_stat = XPRT_DIED;
return (-1);
}
if (cd->nonblock && i != cnt) {
if (cd->nonblock) {
/*
* For non-blocking connections, do not
* take more than 2 seconds writing the
@ -560,6 +560,7 @@ write_vc(xprtp, buf, len)
return (-1);
}
}
i = 0;
}
}