Apart from g_io_channel_unix_new() not setting errno, it never returns an NULL pointer on error, because in practice it only calls g_new(). --- plugins/hfp_hf_bluez5.c | 5 ----- 1 file changed, 5 deletions(-) diff --git a/plugins/hfp_hf_bluez5.c b/plugins/hfp_hf_bluez5.c index 8cac945..ff4dbad 100644 --- a/plugins/hfp_hf_bluez5.c +++ b/plugins/hfp_hf_bluez5.c @@ -137,11 +137,6 @@ static int service_level_connection(struct ofono_modem *modem, GAtChat *chat; io = g_io_channel_unix_new(fd); - if (io == NULL) { - ofono_error("Service level connection failed: %s (%d)", - strerror(errno), errno); - return -EIO; - } syntax = g_at_syntax_new_gsm_permissive(); chat = g_at_chat_new(io, syntax); -- 1.8.1.1