genl_ctrl_probe_by_name: fix checking of genlmsg_put() return value

It's a pointer! not an error code :) nasty bugs

diff --git a/lib/genl/ctrl.c b/lib/genl/ctrl.c
index 59503946204a53728f08b4e1272f577ae42760d3..057a1bfdd4884b9428b9f67c283430e2de7ef7ec 100644 (file)
--- 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;
        }