In the currently uncommon case of core (eg. uverbs) response
space being omitted, but hw (eg. provider) response space being
available, outbuf get defined to "response" while it must be NULL.

This patch takes care of setting ucore.outbuf to NULL
if hdr.out_words is equal to 0.

Cc: Matan Barak <[email protected]>
Signed-off-by: Yann Droneaud <[email protected]>
---

Hi Roland,

This one apply on top of my previous patch to fix sparse warning[1][2]:
rewritting the line make it obvious I missed this rather unseen before
case, but the new improved extensible command scheme allows it,
so it must done right.

[1] http://marc.info/[email protected]
[2] https://patchwork.kernel.org/patch/3202411/

Regards.

 drivers/infiniband/core/uverbs_main.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/infiniband/core/uverbs_main.c 
b/drivers/infiniband/core/uverbs_main.c
index 14d8643..6c4fc63 100644
--- a/drivers/infiniband/core/uverbs_main.c
+++ b/drivers/infiniband/core/uverbs_main.c
@@ -681,7 +681,7 @@ static ssize_t ib_uverbs_write(struct file *filp, const 
char __user *buf,
 
                INIT_UDATA(&ucore,
                           (hdr.in_words) ? buf : NULL,
-                          response,
+                          (hdr.out_words) ? response : NULL,
                           hdr.in_words * 8,
                           hdr.out_words * 8);
 
-- 
1.8.4.2

--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to