Re: [PATCH_v2 2/4] cdma-connman: add username and password, pass them to drivers

[Denis Kenzior <denkenz@gmail.com>]

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

Hi Guillaume,

On 07/20/2011 11:40 PM, Guillaume Zajac wrote:
> ---
>  src/cdma-connman.c |   48 +++++++++++++++++++++++++++++++++++++++++++++++-
>  1 files changed, 47 insertions(+), 1 deletions(-)
> 

I applied this patch, with a follow on commit afterwards:

> @@ -371,6 +373,36 @@ static DBusMessage *cdma_connman_get_properties(DBusConnection *conn,
>  	return reply;
>  }
>  
> +static DBusMessage *cdma_connman_set_username(struct ofono_cdma_connman *cm,
> +					DBusConnection *conn, DBusMessage *msg,
> +					const char *username)
> +{
> +	if (strlen(username) > OFONO_CDMA_CONNMAN_MAX_USERNAME_LENGTH)
> +		return __ofono_error_invalid_format(msg);
> +
> +	if (g_str_equal(username, cm->username))
> +		return dbus_message_new_method_return(msg);
> +
> +	strcpy(cm->username, username);
> +
> +	return dbus_message_new_method_return(msg);

When you're changing a property, you really must signal PropertyChanged
as well.

> +}
> +
> +static DBusMessage *cdma_connman_set_password(struct ofono_cdma_connman *cm,
> +					DBusConnection *conn, DBusMessage *msg,
> +					const char *password)
> +{
> +	if (strlen(password) > OFONO_CDMA_CONNMAN_MAX_PASSWORD_LENGTH)
> +		return __ofono_error_invalid_format(msg);
> +
> +	if (g_str_equal(password, cm->password))
> +		return dbus_message_new_method_return(msg);
> +
> +	strcpy(cm->password, password);
> +
> +	return dbus_message_new_method_return(msg);

same comment as above.

> +}
> +
>  static DBusMessage *cdma_connman_set_property(DBusConnection *conn,
>  					DBusMessage *msg, void *data)
>  {

Regards,
-Denis