Hi,
On Mon, Sep 29, 2014 at 5:57 PM, Luiz Augusto von Dentz
<luiz.dentz@xxxxxxxxx> wrote:
> From: Luiz Augusto von Dentz <luiz.von.dentz@xxxxxxxxx>
>
> Returning -1 normally is used when errno is set which is not case so
> instead return -ENOTCONN.
> ---
> src/shared/io-glib.c | 4 +++-
> src/shared/io-mainloop.c | 3 ++-
> 2 files changed, 5 insertions(+), 2 deletions(-)
>
> diff --git a/src/shared/io-glib.c b/src/shared/io-glib.c
> index 6316037..882ebd6 100644
> --- a/src/shared/io-glib.c
> +++ b/src/shared/io-glib.c
> @@ -25,6 +25,8 @@
> #include <config.h>
> #endif
>
> +#include <errno.h>
> +
> #include <glib.h>
>
> #include "src/shared/io.h"
> @@ -122,7 +124,7 @@ void io_destroy(struct io *io)
> int io_get_fd(struct io *io)
> {
> if (!io)
> - return -1;
> + return -ENOTCONN;
>
> return g_io_channel_unix_get_fd(io->channel);
> }
> diff --git a/src/shared/io-mainloop.c b/src/shared/io-mainloop.c
> index 1563ce5..b7e0f5f 100644
> --- a/src/shared/io-mainloop.c
> +++ b/src/shared/io-mainloop.c
> @@ -26,6 +26,7 @@
> #endif
>
> #include <unistd.h>
> +#include <errno.h>
> #include <sys/socket.h>
>
> #include "monitor/mainloop.h"
> @@ -192,7 +193,7 @@ void io_destroy(struct io *io)
> int io_get_fd(struct io *io)
> {
> if (!io)
> - return -1;
> + return -ENOTCONN;
>
> return io->fd;
> }
> --
> 1.9.3
Pushed.
--
Luiz Augusto von Dentz
--
To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
