Re: [PATCH] modem: do not allow more than one client to lock down

[Marcel Holtmann <marcel@holtmann.org>]

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

Hi Remi,

>  src/modem.c |   13 ++++++-------
>  1 files changed, 6 insertions(+), 7 deletions(-)

patch has been applied.

However I added some paranoia protection for lock_owner reset to NULL. I
know that in theory it is protected via lockdown == TRUE, but that is an
issue bound to break something eventually.

> @@ -847,12 +850,8 @@ static DBusMessage *set_property_lockdown(struct ofono_modem *modem,
>  					OFONO_MODEM_INTERFACE,
>  					"Powered", DBUS_TYPE_BOOLEAN,
>  					&powered);
> -	} else {
> -		if (g_strcmp0(caller, modem->lock_owner))
> -			return __ofono_error_access_denied(msg);
> -
> +	} else
>  		lockdown_remove(modem);
> -	}
>  
>  done:
>  	g_dbus_send_reply(conn, msg, DBUS_TYPE_INVALID);

And since this becomes a one line statement now and we already have a
goto label there, I changed this a bit around and removed the
unnecessary indentation.

Regards

Marcel