diff options
| author | Weston Andros Adamson <dros@netapp.com> | 2013-11-20 12:46:20 -0500 |
|---|---|---|
| committer | Steve Dickson <steved@redhat.com> | 2013-11-20 15:04:47 -0500 |
| commit | d3eac1e59e068cf033f850ab8be016beccf9726d (patch) | |
| tree | 3063079ec2ff87a5a1b9e2faa7a6903b7295e4a9 /utils/gssd/gssd.h | |
| parent | f4b43e2ff3db1b12a60c9b5087ac2bcf6ba4bee5 (diff) | |
| download | nfs-utils-d3eac1e59e068cf033f850ab8be016beccf9726d.tar.gz nfs-utils-d3eac1e59e068cf033f850ab8be016beccf9726d.tar.xz nfs-utils-d3eac1e59e068cf033f850ab8be016beccf9726d.zip | |
gssd: Fix file descriptor leak of old pipe dirs
gssd doesn't properly clean up internal state for old pipes and never
closes the (since deleted) clnt_info directory. This leads to eventual
fd exhaustion.
To reproduce, run a lot of mount / umounts in a loop and watch the
output of 'ls /proc/$PID/fdinfo | wc -l' (where PID is the pid of gssd)
steadily grow until gssd eventually crashes with "Too many open files".
This regression was introduced by 841e83c1, which was trying to fix a
similar bug in the skip matching logic of update_old_clients. The
problem with that patch is that pdir will never match dirname,
because dirname is "<pname>/clntXXX".
Signed-off-by: Weston Andros Adamson <dros@netapp.com>
Signed-off-by: Steve Dickson <steved@redhat.com>
Diffstat (limited to 'utils/gssd/gssd.h')
| -rw-r--r-- | utils/gssd/gssd.h | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/utils/gssd/gssd.h b/utils/gssd/gssd.h index 86472a1..e44ea40 100644 --- a/utils/gssd/gssd.h +++ b/utils/gssd/gssd.h @@ -73,6 +73,7 @@ TAILQ_HEAD(clnt_list_head, clnt_info) clnt_list; struct clnt_info { TAILQ_ENTRY(clnt_info) list; char *dirname; + char *pdir; int dir_fd; char *servicename; char *servername; |