Add virtual destructors to Auth::User hierarchy

diff --git a/src/auth/User.h b/src/auth/User.h
index a2645245e27869d8c1ed2bac1baa603ed0cd204d..e051892c27e8c4f1b0383115b4b2937f4cfa0b1a 100644 (file)
--- a/src/auth/User.h
+++ b/src/auth/User.h
@@ -29,7 +29,6 @@ class Config;
 class CredentialsCache;
 
 /**
- *  \ingroup AuthAPI
  * This is the main user related structure. It stores user-related data,
  * and is persistent across requests. It can even persist across
  * multiple external authentications. One major benefit of preserving this
@@ -41,10 +40,12 @@ class User : public RefCountable
 public:
     typedef RefCount<User> Pointer;
 
+protected:
+    User(Auth::Config *, const char *requestRealm);
+public:
+    virtual ~User();
+
     /* extra fields for proxy_auth */
-    /* auth_type and auth_module are deprecated. Do Not add new users of these fields.
-     * Aim to remove shortly
-     */
     /** \deprecated this determines what scheme owns the user data. */
     Auth::Type auth_type;
     /** the config for this user */
@@ -60,7 +61,6 @@ public:
     static SBuf BuildUserKey(const char *username, const char *realm);
 
     void absorb(Auth::User::Pointer from);
-    virtual ~User();
     char const *username() const { return username_; }
     void username(char const *); ///< set stored username and userKey
 
@@ -100,9 +100,6 @@ private:
      */
     CredentialState credentials_state;
 
-protected:
-    User(Auth::Config *, const char *requestRealm);
-
 private:
     /**
      * DPW 2007-05-08

diff --git a/src/auth/basic/User.h b/src/auth/basic/User.h
index 340e7c8f311169b9e6a363aeaf68b5a7675721b3..74f893f9f53af0ffa8a73ad4d1b202ec6fb08403 100644 (file)
--- a/src/auth/basic/User.h
+++ b/src/auth/basic/User.h
@@ -28,7 +28,7 @@ class User : public Auth::User
 
 public:
     User(Auth::Config *, const char *requestRealm);
-    ~User();
+    virtual ~User();
     bool authenticated() const;
     bool valid() const;
 

diff --git a/src/auth/digest/User.h b/src/auth/digest/User.h
index f823bb5256e0701821e9323b420e490c13a678ff..fc37110f9fd92ca05f076d0ac43fff4dd61c603b 100644 (file)
--- a/src/auth/digest/User.h
+++ b/src/auth/digest/User.h
@@ -25,7 +25,7 @@ class User : public Auth::User
 
 public:
     User(Auth::Config *, const char *requestRealm);
-    ~User();
+    virtual ~User();
     int authenticated() const;
     virtual int32_t ttl() const;
 

diff --git a/src/auth/negotiate/User.h b/src/auth/negotiate/User.h
index 3df41a4eb98123c9c473f0aaf26a7661bb5445b2..3764cbd06be13e7528830203a10ab71970372ba0 100644 (file)
--- a/src/auth/negotiate/User.h
+++ b/src/auth/negotiate/User.h
@@ -26,7 +26,7 @@ class User : public Auth::User
 
 public:
     User(Auth::Config *, const char *requestRealm);
-    ~User();
+    virtual ~User();
     virtual int32_t ttl() const;
 
     /* Auth::User API */

diff --git a/src/auth/ntlm/User.h b/src/auth/ntlm/User.h
index acb2cd724e235d155812f7508aceb9f2ec1df133..57552ac365cb5f5dffbf7eb17098e83e0f2d3c02 100644 (file)
--- a/src/auth/ntlm/User.h
+++ b/src/auth/ntlm/User.h
@@ -26,7 +26,7 @@ class User : public Auth::User
 
 public:
     User(Auth::Config *, const char *requestRealm);
-    ~User();
+    virtual ~User();
     virtual int32_t ttl() const;
 
     /* Auth::User API */