Drop bogus const from headerFreeTag() data pointer
authorPanu Matilainen <pmatilai@redhat.com>
Tue, 18 Dec 2007 08:45:36 +0000 (10:45 +0200)
committerPanu Matilainen <pmatilai@redhat.com>
Tue, 18 Dec 2007 08:45:36 +0000 (10:45 +0200)
- no effect as it's hft is unused
- mark headerFreeData(), the real problem case, with _constfree() for
  easy finding (inlined in header.h so can't use _constfree())

rpmdb/header.c
rpmdb/header.h
rpmdb/header_common.c
rpmdb/header_method.h

index 52c02d6..9a9457b 100644 (file)
@@ -1473,7 +1473,7 @@ static int intGetEntry(Header h, rpm_tag_t tag,
  * @return             NULL always
  */
 static void * _headerFreeTag(Header h,
-               const void * data, rpm_tagtype_t type)
+               void * data, rpm_tagtype_t type)
 {
     if (data) {
        if (type == -1 ||
index 1d49a21..47a01fb 100644 (file)
@@ -395,7 +395,7 @@ int headerIsEntry(Header h, rpm_tag_t tag);
  * @param type         type of data (or -1 to force free)
  * @return             NULL always
  */
-void * headerFreeTag(Header h, const void * data, rpm_tagtype_t type);
+void * headerFreeTag(Header h, void * data, rpm_tagtype_t type);
 
 /** \ingroup header
  * Retrieve tag value.
@@ -590,14 +590,14 @@ int headerNextIterator(HeaderIterator hi,
  * @return             NULL always
  */
 static inline
-void * headerFreeData( const void * data, rpm_tagtype_t type)
+void * headerFreeData(const void * data, rpm_tagtype_t type)
 {
     if (data) {
        if (type == RPM_FORCEFREE_TYPE ||
            type == RPM_STRING_ARRAY_TYPE ||
            type == RPM_I18NSTRING_TYPE ||
            type == RPM_BIN_TYPE)
-               free((void *)data);
+               free((void *)data); /* XXX _constfree() */
     }
     return NULL;
 }
index 20f0123..20ea4dc 100644 (file)
@@ -101,8 +101,7 @@ int headerIsEntry(Header h, rpm_tag_t tag)
     return (h2hv(h)->hdrisentry) (h, tag);
 }
 
-void * headerFreeTag(Header h,
-               const void * data, rpm_tagtype_t type)
+void * headerFreeTag(Header h, void * data, rpm_tagtype_t type)
 {
     return (h2hv(h)->hdrfreetag) (h, data, type);
 }
index b7c6f9f..74a33bf 100644 (file)
@@ -142,8 +142,7 @@ int (*HDRisentry) (Header h, rpm_tag_t tag);
  * @return             NULL always
  */
 typedef
-void * (*HDRfreetag) (Header h,
-               const void * data, rpm_tagtype_t type);
+void * (*HDRfreetag) (Header h, void * data, rpm_tagtype_t type);
 
 /** \ingroup header
  * Retrieve tag value.