diff options
| author | Matthew N. Dodd <matthew.nygard.dodd@gmail.com> | 2011-06-20 13:42:18 -0400 |
|---|---|---|
| committer | Steve Dickson <steved@redhat.com> | 2011-06-20 13:42:18 -0400 |
| commit | a17307911da331e8bf2a0ac94a3a37cc35a06767 (patch) | |
| tree | a7c0c81f4ccc9f3ff4a5f69d77794c7a0e44de30 /tirpc/rpc/auth.h | |
| parent | e716edbbed160423624f6c2f745debf10aad279b (diff) | |
| download | ti-rpc-a17307911da331e8bf2a0ac94a3a37cc35a06767.tar.gz | |
Reference count AUTHs
RPCSEC GSSv3 has the concept of a parent and a compound credential. As
the normal course of operation involves using multiple AUTHs per client
connection, and providing parent and compounds AUTHs when creating a
GSSv3 AUTH, we need a way of reference counting them so that
AUTH_DESTROY does not free them out from under a GSSv3 AUTH that is
using them.
Signed-off-by: Steve Dickson <steved@redhat.com>
Diffstat (limited to 'tirpc/rpc/auth.h')
| -rw-r--r-- | tirpc/rpc/auth.h | 35 |
1 files changed, 31 insertions, 4 deletions
diff --git a/tirpc/rpc/auth.h b/tirpc/rpc/auth.h index f669ae4..5f66e67 100644 --- a/tirpc/rpc/auth.h +++ b/tirpc/rpc/auth.h @@ -203,8 +203,22 @@ typedef struct __auth { } *ah_ops; void *ah_private; + int ah_refcnt; } AUTH; +static __inline int +auth_get(AUTH *auth) +{ + return __sync_add_and_fetch(&auth->ah_refcnt, 1); +} + +static __inline int +auth_put(AUTH *auth) +{ + return __sync_sub_and_fetch(&auth->ah_refcnt, 1); +} + + /* * Authentication ops. @@ -234,10 +248,23 @@ typedef struct __auth { #define auth_refresh(auth, msg) \ ((*((auth)->ah_ops->ah_refresh))(auth, msg)) -#define AUTH_DESTROY(auth) \ - ((*((auth)->ah_ops->ah_destroy))(auth)) -#define auth_destroy(auth) \ - ((*((auth)->ah_ops->ah_destroy))(auth)) +#define AUTH_DESTROY(auth) \ + do { \ + int refs; \ + if ((refs = auth_put((auth))) == 0) \ + ((*((auth)->ah_ops->ah_destroy))(auth));\ + log_debug("%s: auth_put(), refs %d\n", \ + __func__, refs); \ + } while (0) + +#define auth_destroy(auth) \ + do { \ + int refs; \ + if ((refs = auth_put((auth))) == 0) \ + ((*((auth)->ah_ops->ah_destroy))(auth));\ + log_debug("%s: auth_put(), refs %d\n", \ + __func__, refs); \ + } while (0) #define AUTH_WRAP(auth, xdrs, xfunc, xwhere) \ ((*((auth)->ah_ops->ah_wrap))(auth, xdrs, \ |