Hi Gustavo,
On Tue, Oct 18, 2011, Gustavo F. Padovan wrote:
> ---
> audio/avdtp.c | 2 ++
> 1 files changed, 2 insertions(+), 0 deletions(-)
>
> diff --git a/audio/avdtp.c b/audio/avdtp.c
> index d3568ba..08860d2 100644
> --- a/audio/avdtp.c
> +++ b/audio/avdtp.c
> @@ -523,6 +523,8 @@ static gboolean avdtp_send(struct avdtp *session, uint8_t transaction,
> }
>
> sock = g_io_channel_unix_get_fd(session->io);
> + if (!sock)
> + return FALSE;
>
> /* Single packet - no fragmentation */
> if (sizeof(struct avdtp_single_header) + len <= session->omtu) {
You'll need to give some more background when you're seeing this.
session->io should be set to NULL if we don't have a proper socket
available. Also, "if (!sock)" sounds like a weird check for a file
descriptor or socket where 0 is a valid value (typically reserved for
stdin but could get reused for something else in case a daemon-like
process has closed stdin, stdout and stderr). I.e. a validity check for
such a variable should be if (sock < 0), but as I said in this case you
should really investigate why session->io isn't NULL.
Johan
--
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
