Re: [PATCH] emulator: fix dun_ath_cb() not to remove GAtServer io_disconnect() CB

[Guillaume Zajac <guillaume.zajac@linux.intel.com>]

[-- Attachment #1: Type: text/plain, Size: 1321 bytes --]

Hi Denis,

On 19/05/2011 18:11, Denis Kenzior wrote:
> Hi Guillaume,
>
> On 05/19/2011 08:03 AM, Guillaume Zajac wrote:
>> ---
>>   src/emulator.c |    4 ++++
>>   1 files changed, 4 insertions(+), 0 deletions(-)
>>
>> diff --git a/src/emulator.c b/src/emulator.c
>> index c17b901..251bed8 100644
>> --- a/src/emulator.c
>> +++ b/src/emulator.c
>> @@ -219,15 +219,19 @@ static void dun_ath_cb(GAtServer *server, GAtServerRequestType type,
>>   		if (val != 0)
>>   			goto error;
>>
>> +		g_at_server_suspend(em->server);
>>   		g_at_ppp_unref(em->ppp);
>>   		em->ppp = NULL;
>> +		g_at_server_resume(em->server);
>>
>>   		g_at_server_send_final(server, G_AT_SERVER_RESULT_OK);
>>   		break;
>>
>>   	case G_AT_SERVER_REQUEST_TYPE_COMMAND_ONLY:
>> +		g_at_server_suspend(em->server);
>>   		g_at_ppp_unref(em->ppp);
>>   		em->ppp = NULL;
>> +		g_at_server_resume(em->server);
>>
>>   		g_at_server_send_final(server, G_AT_SERVER_RESULT_OK);
>>   		break;
> This really sounds wrong.  I suggest that we not set the disconnect
> function in GAtPPP when unrefing if GAtPPP is currently suspended.

Ok, I will do it like this.
When I will submit the resume functions associated to suspend functions, 
I will also have to take care about suspended state.

Kind regards,
Guillaume