From a4990946d6ae42a4f66439acca63668c61690990 Mon Sep 17 00:00:00 2001 From: Marcel Holtmann Date: Sun, 4 Mar 2012 20:34:19 -0800 Subject: core: Add extra checks for set property methods --- src/clock.c | 7 +++++++ 1 file changed, 7 insertions(+) (limited to 'src/clock.c') diff --git a/src/clock.c b/src/clock.c index 0d7f8705..c3644ea9 100644 --- a/src/clock.c +++ b/src/clock.c @@ -169,8 +169,15 @@ static DBusMessage *set_property(DBusConnection *conn, if (dbus_message_iter_init(msg, &iter) == FALSE) return __connman_error_invalid_arguments(msg); + if (dbus_message_iter_get_arg_type(&iter) != DBUS_TYPE_STRING) + return __connman_error_invalid_arguments(msg); + dbus_message_iter_get_basic(&iter, &name); dbus_message_iter_next(&iter); + + if (dbus_message_iter_get_arg_type(&iter) != DBUS_TYPE_VARIANT) + return __connman_error_invalid_arguments(msg); + dbus_message_iter_recurse(&iter, &value); type = dbus_message_iter_get_arg_type(&value); -- cgit v1.2.3