Re: [PATCH obexd] gobex: call g_obex_unref() before io channel unref

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Luiz,

--------------------------------------------------
From: "Luiz Augusto von Dentz" <luiz.dentz@xxxxxxxxx>
Sent: Thursday, March 08, 2012 5:58 PM
To: <jaganath.k@xxxxxxxxxxx>
Cc: <linux-bluetooth@xxxxxxxxxxxxxxx>
Subject: Re: [PATCH obexd] gobex: call g_obex_unref() before io channel unref

Hi Jaganath,

On Thu, Mar 8, 2012 at 8:54 AM, JAGANATH KANAKKASSERY
<jaganath.k@xxxxxxxxxxx> wrote:

Hi

------- Original Message -------
Sender : JAGANATH KANAKKASSERY<jaganath.k@xxxxxxxxxxx> Lead Engineer/SISO-Linux Platform Lab/Samsung Electronics
Date   : Mar 08, 2012 15:48 (GMT+09:00)
Title  : [PATCH obexd] gobex: call g_obex_unref() before io channel unref

In g_obex_unref, tx_queue will be flushed which results in SIGPIPE
in g_io_channel_write_chars() since socket is already closed in
io channel unref.

Please ignore this patch

I guess if the io is closed you don't need to flush anything, in the
other hand it would be good to add a test that does what PTS expect
with abort being sent.

I have raised the the same patch with subject  [PATCH obexd 2/2]. This
patch is required to pass unit test after applying my first patch
[PATCH obexd 1/2] gobex: flush tx_queue before disconnection
Otherwise while trying to access io in g_io_channel_write_chars()
SIGPIPE is coming which kills the process.

--
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

--
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


[Index of Archives]     [Bluez Devel]     [Linux Wireless Networking]     [Linux Wireless Personal Area Networking]     [Linux ATH6KL]     [Linux USB Devel]     [Linux Media Drivers]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Big List of Linux Books]

  Powered by Linux