libvhost-user: fix SLAVE_SEND_FD handling
authorJohannes Berg <johannes.berg@intel.com>
Tue, 3 Sep 2019 20:04:22 +0000 (23:04 +0300)
committerMichael S. Tsirkin <mst@redhat.com>
Wed, 4 Sep 2019 11:23:42 +0000 (07:23 -0400)
commit8726b70b449896f1211f869ec4f608904f027207
treec958227b13d4143cf5f58f1a02f7bcd9355e8b28
parenteb1556c493d8abc5bfc8685561bcea934700e200
libvhost-user: fix SLAVE_SEND_FD handling

It doesn't look like this could possibly work properly since
VHOST_USER_PROTOCOL_F_SLAVE_SEND_FD is defined to 10, but the
dev->protocol_features has a bitmap. I suppose the peer this
was tested with also supported VHOST_USER_PROTOCOL_F_LOG_SHMFD,
in which case the test would always be false, but nevertheless
the code seems wrong.

Use has_feature() to fix this.

Fixes: d84599f56c82 ("libvhost-user: support host notifier")
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Message-Id: <20190903200422.11693-1-johannes@sipsolutions.net>
Reviewed-by: Tiwei Bie <tiwei.bie@intel.com>
Reviewed-by: Michael S. Tsirkin <mst@redhat.com>
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
contrib/libvhost-user/libvhost-user.c