| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
| |
the passed in auth_type is unacceptable to rpcauth_buildheader-
this avoids a null pointer panic. Clean up allocations if this
happens. This also quiets a gcc 4.2 complaint about ussing mheadend
without it being initialized.
Reviewed by: alfred
|
|
|
|
|
|
|
| |
If/when someone does the necessary MPSAFEty locking for the NFSv4 client,
the socket code is generally MPSAFE now.
Spotted by: kris
|
|
|
|
|
|
| |
No code changes.
Submitted by: cel@citi.umich.edu
|
|
|
|
|
|
|
|
|
| |
This fixes reconnect after, for example, tcp idle disconnection.
Previously this would fail if a normal user tried to bind to a privileged
port.
Submitted by: cel@citi.umich.edu
MFC after: 1 week
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
as they both interact with the tty code (!MPSAFE) and may sleep if the
tty buffer is full (per comment).
Modify all consumers of uprintf() and tprintf() to hold Giant around
calls into these functions. In most cases, this means adding an
acquisition of Giant immediately around the function. In some cases
(nfs_timer()), it means acquiring Giant higher up in the callout.
With these changes, UFS no longer panics on SMP when either blocks are
exhausted or inodes are exhausted under load due to races in the tty
code when running without Giant.
NB: Some reduction in calls to uprintf() in the svr4 code is probably
desirable.
NB: In the case of nfs_timer(), calling uprintf() while holding a mutex,
or even in a callout at all, is a bad idea, and will generate warnings
and potential upset. This needs to be fixed, but was a problem before
this change.
NB: uprintf()/tprintf() sleeping is generally a bad ideas, as is having
non-MPSAFE tty code.
MFC after: 1 week
|
|
|
|
|
|
|
| |
- Remove a bogus and unneeded null pointer check.
Found by: Coverity Prevent analysis tool
Approved by: alfred
|
| |
|
|
|
|
|
| |
Noticed by: tedu
Approved by: alfred
|
| |
|
|
|
|
|
|
|
|
|
| |
are supposed to continue firing as long as there is work to do, not
stop after the first invocation.
This is damage control after a patch that has been committed prematurely.
Tested by: kris
|
|
|
|
|
| |
Submitted by: kan
Approved by: alfred
|
| |
|
|
|
|
| |
Submitted by: Jim Rees <rees@umich.edu>
|
|
|
|
|
| |
Submitted by: Jim Rees <rees@umich.edu>
Reported/testing: Florian C. Smeets <flo@kasimir.com>
|
|
|
|
|
|
|
| |
It happens because rpcclnt_request is incorrectly returning 0 in the case
of an rpc mismatch or auth error.
Submitted by: Jim Rees <rees@umich.edu>
|
|
|
|
| |
Requested by: jmallett, wollman
|
|
|
|
| |
Cast sizeof to int for printing with %d.
|
|
Submitted by: Jim Rees <rees@umich.edu>
|