util/disk_cache: stop using ralloc_asprintf() unnecessarily
authorTimothy Arceri <tarceri@itsqueeze.com>
Wed, 8 Feb 2017 22:04:52 +0000 (09:04 +1100)
committerTimothy Arceri <tarceri@itsqueeze.com>
Thu, 9 Feb 2017 03:11:24 +0000 (14:11 +1100)
Reviewed-by: Anuj Phogat <anuj.phogat@gmail.com>
src/util/disk_cache.c

index 7451b08..8853236 100644 (file)
@@ -336,11 +336,12 @@ static char *
 get_cache_file(struct disk_cache *cache, cache_key key)
 {
    char buf[41];
+   char *filename;
 
    _mesa_sha1_format(buf, key);
+   asprintf(&filename, "%s/%c%c/%s", cache->path, buf[0], buf[1], buf + 2);
 
-   return ralloc_asprintf(cache, "%s/%c%c/%s",
-                          cache->path, buf[0], buf[1], buf + 2);
+   return filename;
 }
 
 /* Create the directory that will be needed for the cache file for \key.
@@ -355,12 +356,10 @@ make_cache_file_directory(struct disk_cache *cache, cache_key key)
    char buf[41];
 
    _mesa_sha1_format(buf, key);
-
-   dir = ralloc_asprintf(cache, "%s/%c%c", cache->path, buf[0], buf[1]);
-
+   asprintf(&dir, "%s/%c%c", cache->path, buf[0], buf[1]);
    mkdir_if_needed(dir);
 
-   ralloc_free(dir);
+   free(dir);
 }
 
 /* Given a directory path and predicate function, count all entries in
@@ -548,12 +547,12 @@ disk_cache_remove(struct disk_cache *cache, cache_key key)
    }
 
    if (stat(filename, &sb) == -1) {
-      ralloc_free(filename);
+      free(filename);
       return;
    }
 
    unlink(filename);
-   ralloc_free(filename);
+   free(filename);
 
    if (sb.st_size)
       p_atomic_add(cache->size, - sb.st_size);
@@ -578,7 +577,7 @@ disk_cache_put(struct disk_cache *cache,
     * final destination filename, (to prevent any readers from seeing
     * a partially written file).
     */
-   filename_tmp = ralloc_asprintf(cache, "%s.tmp", filename);
+   asprintf(&filename_tmp, "%s.tmp", filename);
    if (filename_tmp == NULL)
       goto done;
 
@@ -650,9 +649,9 @@ disk_cache_put(struct disk_cache *cache,
    if (fd != -1)
       close(fd);
    if (filename_tmp)
-      ralloc_free(filename_tmp);
+      free(filename_tmp);
    if (filename)
-      ralloc_free(filename);
+      free(filename);
 }
 
 void *
@@ -687,7 +686,7 @@ disk_cache_get(struct disk_cache *cache, cache_key key, size_t *size)
          goto fail;
    }
 
-   ralloc_free(filename);
+   free(filename);
    close(fd);
 
    if (size)
@@ -699,7 +698,7 @@ disk_cache_get(struct disk_cache *cache, cache_key key, size_t *size)
    if (data)
       free(data);
    if (filename)
-      ralloc_free(filename);
+      free(filename);
    if (fd != -1)
       close(fd);