IB/uverbs: New macro to set pointers to NULL if length is 0 in INIT_UDATA()

Browse source

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>

...

This commit is contained in:

Roland Dreier 2013-12-19 08:37:03 -08:00

parent 309243ec14

commit a96e4e2ffe

2 changed files with 16 additions and 11 deletions

Show all changes Ignore whitespace when comparing lines Ignore changes in amount of whitespace Ignore changes in whitespace at EOL

Download patch file Download diff file Expand all files Collapse all files

8

drivers/infiniband/core/uverbs.h

View file

17

drivers/infiniband/core/uverbs_main.c

View file