diff options
| author | Коренберг Марк <mark@ideco.ru> | 2012-08-30 22:53:19 +0600 |
|---|---|---|
| committer | Коренберг Марк <mark@ideco.ru> | 2012-08-30 22:53:54 +0600 |
| commit | 49c94c3490fe2691293a8d45b3dab574ee73ca31 (patch) | |
| tree | c15af8e61680cf124ec5dcb92799c2d058f4c985 | |
| parent | 00261259f02b41b0689c87525fd5b6039b651c37 (diff) | |
| download | libnl-49c94c3490fe2691293a8d45b3dab574ee73ca31.tar.gz | |
genl_ctrl_probe_by_name: fix checking of genlmsg_put() return value
It's a pointer! not an error code :) nasty bugs
| -rw-r--r-- | lib/genl/ctrl.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/lib/genl/ctrl.c b/lib/genl/ctrl.c index 5950394..057a1bf 100644 --- a/lib/genl/ctrl.c +++ b/lib/genl/ctrl.c @@ -255,8 +255,8 @@ static struct genl_family *genl_ctrl_probe_by_name(struct nl_sock *sk, if (!(cb = nl_cb_clone(nl_socket_get_cb(sk)))) goto out_msg_free; - if (genlmsg_put(msg, NL_AUTO_PORT, NL_AUTO_SEQ, GENL_ID_CTRL, - 0, 0, CTRL_CMD_GETFAMILY, 1) < 0) { + if (!genlmsg_put(msg, NL_AUTO_PORT, NL_AUTO_SEQ, GENL_ID_CTRL, + 0, 0, CTRL_CMD_GETFAMILY, 1)) { BUG(); goto out_cb_free; } |