Return-Path: Date: Tue, 26 Apr 2011 15:41:41 -0300 From: Vinicius Costa Gomes To: "Zheng, Wu" Cc: "linux-bluetooth@vger.kernel.org" Subject: Re: [PATCH] Check session->msg at sync_getphonebook_callback() in file client/sync.c Message-ID: <20110426184141.GA2653@piper> References: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: Sender: linux-bluetooth-owner@vger.kernel.org List-ID: Hi, On 11:03 Tue 26 Apr, Zheng, Wu wrote: > Otherwise,session->msg will be NULL and cause segmentation fault error.The reason is that the condition of transfer->fd>0 && session->msg is ok and session->msg will be set to NULL in the func of session_notify_progress when sync profile is used. Please try to keep your commit title to less than 72 characters (if possible less than 50) and the commit message to less than 72 characters. > > --- > client/sync.c | 5 +++++ > 1 files changed, 5 insertions(+), 0 deletions(-) > > diff --git a/client/sync.c b/client/sync.c > index 3622a3d..28ace61 100644 > --- a/client/sync.c > +++ b/client/sync.c > @@ -78,6 +78,9 @@ static void sync_getphonebook_callback(struct session_data *session, > DBusMessage *reply; > char *buf = NULL; > > + if (session->msg == NULL) > + goto done; > + > reply = dbus_message_new_method_return(session->msg); > > if (transfer->filled > 0) > @@ -91,6 +94,8 @@ static void sync_getphonebook_callback(struct session_data *session, > g_dbus_send_message(session->conn, reply); > dbus_message_unref(session->msg); > session->msg = NULL; Add an empty line here. > +done: > + transfer_unregister(transfer); There's a trailing space here. Also, taking a look at the code, if the reply is big enough (is it possible?), the callback could be called more than one time for one transfer, right? Seems it would cause some unexpected problems. > } > > static DBusMessage *sync_getphonebook(DBusConnection *connection, > -- > 1.7.3.4 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html Cheers, -- Vinicius