lib/lock: loose the internal const

The internal const lead to a cast in every destroy function.
This crept in early on and hasn't been cleaned out. Do so now.

Signed-off-by: Marcelo Vanzin <mvanzin@vmware.com>

diff --git a/open-vm-tools/lib/lock/ulBarrier.c b/open-vm-tools/lib/lock/ulBarrier.c
index 6591d7bf572f65529a3cd3585aa29d67dab130ee..407629b4bd01a372ede79a00cc9e805e19b60553 100644 (file)
--- a/open-vm-tools/lib/lock/ulBarrier.c
+++ b/open-vm-tools/lib/lock/ulBarrier.c
@@ -203,7 +203,7 @@ MXUser_DestroyBarrier(MXUserBarrier *barrier)  // IN:
       MXUser_DestroyExclLock(barrier->lock);
 
       barrier->header.signature = 0;  // just in case...
-      free((void *) barrier->header.name);  // avoid const warnings
+      free(barrier->header.name);
       barrier->header.name = NULL;
       free(barrier);
    }

diff --git a/open-vm-tools/lib/lock/ulExcl.c b/open-vm-tools/lib/lock/ulExcl.c
index 99d1475124d55f66e63f2c5a233fa158434348ea..48a155ce07b688b08b168428507c7e8e7016f969 100644 (file)
--- a/open-vm-tools/lib/lock/ulExcl.c
+++ b/open-vm-tools/lib/lock/ulExcl.c
@@ -262,7 +262,7 @@ MXUser_DestroyExclLock(MXUserExclLock *lock)  // IN:
       }
 
       lock->header.signature = 0;  // just in case...
-      free((void *) lock->header.name);  // avoid const warnings
+      free(lock->header.name);
       lock->header.name = NULL;
       free(lock);
    }

diff --git a/open-vm-tools/lib/lock/ulInt.h b/open-vm-tools/lib/lock/ulInt.h
index 9ab7cc2388ae43d47616663e93485ec740a33738..415718687e34e957bb0275096944eb8abee495f5 100644 (file)
--- a/open-vm-tools/lib/lock/ulInt.h
+++ b/open-vm-tools/lib/lock/ulInt.h
@@ -416,7 +416,7 @@ MXUserGetNativeTID(void)
 typedef struct MXUserHeader {
    uint32       signature;
    MX_Rank      rank;
-   const char  *name;
+   char        *name;
    uint32       identifier;
    void       (*dumpFunc)(struct MXUserHeader *);
 

diff --git a/open-vm-tools/lib/lock/ulRW.c b/open-vm-tools/lib/lock/ulRW.c
index 85d2e9d8affc52c5a2908c161a8a1edc007e01df..ffc3f5f701af21a006b561d0e57753461ee81d1a 100644 (file)
--- a/open-vm-tools/lib/lock/ulRW.c
+++ b/open-vm-tools/lib/lock/ulRW.c
@@ -31,6 +31,12 @@
 
 #define MXUSER_RW_SIGNATURE 0x57524B4C // 'LKRW' in memory
 
+static void
+MXUserFreeHashEntry(void *data)  // IN:
+{
+   free(data);
+}
+
 
 /*
  * Environment specific implementations of portable read-write locks.
@@ -397,12 +403,6 @@ MXUserDumpRWLock(MXUserHeader *header)  // IN:
  *-----------------------------------------------------------------------------
  */
 
-static void
-MXUserFreeHashEntry(void *data)  // IN:
-{
-   free(data);
-}
-
 MXUserRWLock *
 MXUser_CreateRWLock(const char *userName,  // IN:
                     MX_Rank rank)          // IN:
@@ -544,7 +544,7 @@ MXUser_DestroyRWLock(MXUserRWLock *lock)  // IN:
 
       HashTable_FreeUnsafe(lock->holderTable);
       lock->header.signature = 0;  // just in case...
-      free((void *) lock->header.name);  // avoid const warnings
+      free(lock->header.name);
       lock->header.name = NULL;
       free(lock);
    }

diff --git a/open-vm-tools/lib/lock/ulRec.c b/open-vm-tools/lib/lock/ulRec.c
index d9ac961a8166f44e5020cf4641f31b515f295327..819617c952e61e9f02b9140fca4895dbd0af6eed 100644 (file)
--- a/open-vm-tools/lib/lock/ulRec.c
+++ b/open-vm-tools/lib/lock/ulRec.c
@@ -358,7 +358,7 @@ MXUser_DestroyRecLock(MXUserRecLock *lock)  // IN:
       }
 
       lock->header.signature = 0;  // just in case...
-      free((void *) lock->header.name);  // avoid const warnings
+      free(lock->header.name);
       lock->header.name = NULL;
       free(lock);
    }

diff --git a/open-vm-tools/lib/lock/ulSema.c b/open-vm-tools/lib/lock/ulSema.c
index 075e0d9242374467ae02960ae00648af7ff8e56d..fd467691f97db4fd40bed1cffdb93c31dfaea68b 100644 (file)
--- a/open-vm-tools/lib/lock/ulSema.c
+++ b/open-vm-tools/lib/lock/ulSema.c
@@ -593,7 +593,7 @@ MXUser_DestroySemaphore(MXUserSemaphore *sema)  // IN:
       }
 
       sema->header.signature = 0;  // just in case...
-      free((void *) sema->header.name);  // avoid const warnings
+      free(sema->header.name);
       sema->header.name = NULL;
       free(sema);
    }