diff options
author | Martin Pitt <martin.pitt@ubuntu.com> | 2016-10-17 07:12:20 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2016-10-17 07:12:20 +0200 |
commit | a7753693547233e4a1d6e10b1a8f6515a477f227 (patch) | |
tree | 3b20f53b931e953f4ed495bb10e90597f3670237 /src/cryptsetup/cryptsetup-generator.c | |
parent | 9e7727521ff2e751ee064716ab735016abf72025 (diff) | |
parent | 3b319885c4febb5f7ea9b5ab31c3395548ed6886 (diff) |
Merge pull request #4391 from keszybz/treewide-macros
Use mfree more and add another function to simplify a common set&free pattern
Diffstat (limited to 'src/cryptsetup/cryptsetup-generator.c')
-rw-r--r-- | src/cryptsetup/cryptsetup-generator.c | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/src/cryptsetup/cryptsetup-generator.c b/src/cryptsetup/cryptsetup-generator.c index 8ac5ab730a..de0a3b6f9c 100644 --- a/src/cryptsetup/cryptsetup-generator.c +++ b/src/cryptsetup/cryptsetup-generator.c @@ -264,16 +264,13 @@ static crypto_device *get_crypto_device(const char *uuid) { d->keyfile = d->options = d->name = NULL; d->uuid = strdup(uuid); - if (!d->uuid) { - free(d); - return NULL; - } + if (!d->uuid) + return mfree(d); r = hashmap_put(arg_disks, d->uuid, d); if (r < 0) { free(d->uuid); - free(d); - return NULL; + return mfree(d); } } |