diff options
| author | Thorsten Blum <thorsten.blum@linux.dev> | 2025-11-12 18:14:07 +0100 |
|---|---|---|
| committer | Juergen Gross <jgross@suse.com> | 2025-11-17 07:55:10 +0100 |
| commit | 6fec913ff1c9811e35e1bb64779efce9bb41b7ec (patch) | |
| tree | fc2c3a4ecdcb0f45a8fc414ea185a1571239909d | |
| parent | 6a23ae0a96a600d1d12557add110e0bb6e32730c (diff) | |
drivers/xen/xenbus: Simplify return statement in join()
Don't unnecessarily negate 'buffer' and simplify the return statement.
Reviewed-by: Jason Andryuk <jason.andryuk@amd.com>
Signed-off-by: Thorsten Blum <thorsten.blum@linux.dev>
Signed-off-by: Juergen Gross <jgross@suse.com>
Message-ID: <20251112171410.3140-2-thorsten.blum@linux.dev>
| -rw-r--r-- | drivers/xen/xenbus/xenbus_xs.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/xen/xenbus/xenbus_xs.c b/drivers/xen/xenbus/xenbus_xs.c index 528682bf0c7f..7c6c12925326 100644 --- a/drivers/xen/xenbus/xenbus_xs.c +++ b/drivers/xen/xenbus/xenbus_xs.c @@ -407,7 +407,7 @@ static char *join(const char *dir, const char *name) buffer = kasprintf(GFP_NOIO | __GFP_HIGH, "%s", dir); else buffer = kasprintf(GFP_NOIO | __GFP_HIGH, "%s/%s", dir, name); - return (!buffer) ? ERR_PTR(-ENOMEM) : buffer; + return buffer ?: ERR_PTR(-ENOMEM); } static char **split(char *strings, unsigned int len, unsigned int *num) |