Re: [PATCH 3/6] android/socket: Handle Android events for server socket

[Andrei Emeltchenko <Andrei.Emeltchenko.news@gmail.com>]

Hi Johan,

On Tue, Nov 26, 2013 at 05:46:02PM +0200, Johan Hedberg wrote:
> Hi Andrei,
> 
> On Tue, Nov 26, 2013, Andrei Emeltchenko wrote:
> > Add watch for tracking events from Android framework for server socket.
> > ---
> >  android/socket.c | 27 ++++++++++++++++++++++++++-
> >  1 file changed, 26 insertions(+), 1 deletion(-)
> 
> I've applied the first two patches, but wanted to ask about this one:
> 
> > +static gboolean sock_server_stack_event_cb(GIOChannel *io, GIOCondition cond,
> > +								gpointer data)
> > +{
> > +	struct rfcomm_sock *rfsock = data;
> > +
> > +	DBG("");
> > +
> > +	if (cond & (G_IO_ERR | G_IO_HUP | G_IO_NVAL)) {
> > +		error("Socket error: sock %d cond %d",
> > +					g_io_channel_unix_get_fd(io), cond);
> > +		cleanup_rfsock(rfsock);
> > +
> > +		return FALSE;
> > +	}
> > +
> > +	return TRUE;
> > +}
> 
> I don't see where (in which patch) you'd add code to handle G_IO_IN on
> this socket. Aren't you supposed to read data from this socket and write
> it to the RFCOMM one?

At this moment I do not know which data might come from this socket. I
have debug statement to know that something is coming.

I assume this would be primary used to clean up socket structure if
Android decides to stop listen().

Best regards 
Andrei Emeltchenko