diff options
| author | Felipe Franciosi | 2016-09-29 17:52:36 +0200 |
|---|---|---|
| committer | Daniel P. Berrange | 2016-10-26 18:19:53 +0200 |
| commit | e413ae0c0492c10d9277a1155ecc21fbbf0e2bc7 (patch) | |
| tree | 3f1fbda4a2ed68545f632380e6b1864bbac06b00 /io/channel-socket.c | |
| parent | io: Fix double shift usages on QIOChannel features (diff) | |
| download | qemu-e413ae0c0492c10d9277a1155ecc21fbbf0e2bc7.tar.gz qemu-e413ae0c0492c10d9277a1155ecc21fbbf0e2bc7.tar.xz qemu-e413ae0c0492c10d9277a1155ecc21fbbf0e2bc7.zip | |
io: Use qio_channel_has_feature() where applicable
Parts of the code have been testing QIOChannel features directly with a
logical AND. This patch makes it all consistent by using the
qio_channel_has_feature() function to test if a feature is present.
Signed-off-by: Felipe Franciosi <felipe@nutanix.com>
Signed-off-by: Daniel P. Berrange <berrange@redhat.com>
Diffstat (limited to 'io/channel-socket.c')
| -rw-r--r-- | io/channel-socket.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/io/channel-socket.c b/io/channel-socket.c index 6710b2ee96..8fc6e5aaaa 100644 --- a/io/channel-socket.c +++ b/io/channel-socket.c @@ -403,7 +403,8 @@ static void qio_channel_socket_finalize(Object *obj) QIOChannelSocket *ioc = QIO_CHANNEL_SOCKET(obj); if (ioc->fd != -1) { - if (QIO_CHANNEL(ioc)->features & (1 << QIO_CHANNEL_FEATURE_LISTEN)) { + QIOChannel *ioc_local = QIO_CHANNEL(ioc); + if (qio_channel_has_feature(ioc_local, QIO_CHANNEL_FEATURE_LISTEN)) { Error *err = NULL; socket_listen_cleanup(ioc->fd, &err); |