diff options
| author |   Roland Dreier <roland@purestorage.com> | 2013-12-19 08:37:03 -0800 |
|---|---|---|
| committer | Roland Dreier <roland@purestorage.com> | 2013-12-20 10:53:44 -0800 |
| commit | a96e4e2ffe439e45732d820fac6fee486b6412bf (patch) | |
| tree | 94b6871a378f9a643f268c1e5c6ccd563fd65fa6 /drivers/infiniband/core/uverbs.h | |
| parent | IB/core: const'ify inbuf in struct ib_udata (diff) | |
| download | linux-dev-a96e4e2ffe439e45732d820fac6fee486b6412bf.tar.xz linux-dev-a96e4e2ffe439e45732d820fac6fee486b6412bf.zip | |
IB/uverbs: New macro to set pointers to NULL if length is 0 in INIT_UDATA()
Trying to have a ternary operator to choose between NULL (or 0) and the
real pointer value in invocations leads to an impossible choice between
a sparse error about a literal 0 used as a NULL pointer, and a gcc
warning about "pointer/integer type mismatch in conditional expression."
Rather than clutter the source with more casts, move the ternary
operator into a new INIT_UDATA_BUF_OR_NULL() macro, which makes it
easier to use and simplifies its callers.
Reported-by: Yann Droneaud <ydroneaud@opteya.com>
Signed-off-by: Roland Dreier <roland@purestorage.com>
Diffstat (limited to 'drivers/infiniband/core/uverbs.h')
| -rw-r--r-- | drivers/infiniband/core/uverbs.h | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/drivers/infiniband/core/uverbs.h b/drivers/infiniband/core/uverbs.h index 9879568aed8c..a283274a5a09 100644 --- a/drivers/infiniband/core/uverbs.h +++ b/drivers/infiniband/core/uverbs.h @@ -55,6 +55,14 @@ (udata)->outlen = (olen); \ } while (0) +#define INIT_UDATA_BUF_OR_NULL(udata, ibuf, obuf, ilen, olen) \ + do { \ + (udata)->inbuf = (ilen) ? (const void __user *) (ibuf) : NULL; \ + (udata)->outbuf = (olen) ? (void __user *) (obuf) : NULL; \ + (udata)->inlen = (ilen); \ + (udata)->outlen = (olen); \ + } while (0) + /* * Our lifetime rules for these structs are the following: * |