2013-07-17 03:15:18

by martin.xu

[permalink] [raw]
Subject: [PATCH] obexd/session: Do not check session->driver->uuid in get_target()

From: Martin Xu <[email protected]>

since it already be checked at target_exists(), so if session->driver->uuid
is NULL get_target() should not be called.
---
obexd/client/session.c | 3 ---
1 file changed, 3 deletions(-)

diff --git a/obexd/client/session.c b/obexd/client/session.c
index 44e2bf8..361c921 100644
--- a/obexd/client/session.c
+++ b/obexd/client/session.c
@@ -680,9 +680,6 @@ static gboolean get_target(const GDBusPropertyTable *property,
{
struct obc_session *session = data;

- if (session->driver->uuid == NULL)
- return FALSE;
-
dbus_message_iter_append_basic(iter, DBUS_TYPE_STRING,
&session->driver->uuid);

--
1.7.10.4



2013-07-17 08:17:37

by Luiz Augusto von Dentz

[permalink] [raw]
Subject: Re: [PATCH] obexd/session: Do not check session->driver->uuid in get_target()

Hi Martin,

On Wed, Jul 17, 2013 at 6:15 AM, <[email protected]> wrote:
> From: Martin Xu <[email protected]>
>
> since it already be checked at target_exists(), so if session->driver->uuid
> is NULL get_target() should not be called.
> ---
> obexd/client/session.c | 3 ---
> 1 file changed, 3 deletions(-)
>
> diff --git a/obexd/client/session.c b/obexd/client/session.c
> index 44e2bf8..361c921 100644
> --- a/obexd/client/session.c
> +++ b/obexd/client/session.c
> @@ -680,9 +680,6 @@ static gboolean get_target(const GDBusPropertyTable *property,
> {
> struct obc_session *session = data;
>
> - if (session->driver->uuid == NULL)
> - return FALSE;
> -
> dbus_message_iter_append_basic(iter, DBUS_TYPE_STRING,
> &session->driver->uuid);
>
> --
> 1.7.10.4

This is used as a safe guard to avoid any chance that NULL is passed
to dbus_message_iter_append_basic in case get_target is called from
another context, it isn't the case here but it is consistent with
other parts of the code.


--
Luiz Augusto von Dentz