diff options
| author | Andrea Claudi <aclaudi@redhat.com> | 2021-04-18 14:56:30 +0200 |
|---|---|---|
| committer | Stephen Hemminger <stephen@networkplumber.org> | 2021-04-20 18:08:38 -0700 |
| commit | c8216fabe8d9df3db38283cca1b6caeca033f9b9 (patch) | |
| tree | 1a1f1c4d68b49b55134ea9b4ad3561b1890a5610 /rdma | |
| parent | 16ce4d23661aee6cecfea0a8ded0ce11ef3a3ecd (diff) | |
| download | iproute2-c8216fabe8d9df3db38283cca1b6caeca033f9b9.tar.gz | |
rdma: stat: fix return code
libmnl defines MNL_CB_OK as 1 and MNL_CB_ERROR as -1. rdma uses these
return codes, and stat_qp_show_parse_cb() should do the same.
Fixes: 16ce4d23661a ("rdma: stat: initialize ret in stat_qp_show_parse_cb()")
Reported-by: Leon Romanovsky <leon@kernel.org>
Signed-off-by: Andrea Claudi <aclaudi@redhat.com>
Acked-by: Leon Romanovsky <leonro@nvidia.com>
Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
Diffstat (limited to 'rdma')
| -rw-r--r-- | rdma/stat.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/rdma/stat.c b/rdma/stat.c index 3abedae7..8edf7bf1 100644 --- a/rdma/stat.c +++ b/rdma/stat.c @@ -307,7 +307,7 @@ static int stat_qp_show_parse_cb(const struct nlmsghdr *nlh, void *data) struct rd *rd = data; const char *name; uint32_t idx; - int ret = 0; + int ret = MNL_CB_OK; mnl_attr_parse(nlh, 0, rd_attr_cb, tb); if (!tb[RDMA_NLDEV_ATTR_DEV_INDEX] || !tb[RDMA_NLDEV_ATTR_DEV_NAME] || |