chiark / gitweb /
memfd: use _cleanup_ if applicable
authorDavid Herrmann <dh.herrmann@gmail.com>
Mon, 18 Aug 2014 11:07:43 +0000 (13:07 +0200)
committerDavid Herrmann <dh.herrmann@gmail.com>
Mon, 18 Aug 2014 11:07:43 +0000 (13:07 +0200)
We now have a sd_memfd_freep helper, use it if applicable.

src/shared/memfd.c

index d94c6268ddf5e376df3ea29956b22b4c1ce8d9b7..e246f915ca9a1cb2ca19c1bb5cbe48ff3e9e2dc9 100644 (file)
@@ -241,7 +241,7 @@ int sd_memfd_set_size(sd_memfd *m, uint64_t sz) {
 }
 
 int sd_memfd_new_and_map(sd_memfd **m, const char *name, size_t sz, void **p) {
-        sd_memfd *n;
+        _cleanup_(sd_memfd_freep) sd_memfd *n = NULL;
         int r;
 
         r = sd_memfd_new(&n, name);
@@ -249,18 +249,15 @@ int sd_memfd_new_and_map(sd_memfd **m, const char *name, size_t sz, void **p) {
                 return r;
 
         r = sd_memfd_set_size(n, sz);
-        if (r < 0) {
-                sd_memfd_free(n);
+        if (r < 0)
                 return r;
-        }
 
         r = sd_memfd_map(n, 0, sz, p);
-        if (r < 0) {
-                sd_memfd_free(n);
+        if (r < 0)
                 return r;
-        }
 
         *m = n;
+        n = NULL;
         return 0;
 }