From 282ba6561e078b533ed4a38b77a36e23181c04ca Mon Sep 17 00:00:00 2001 From: Dan Williams Date: Mon, 22 Mar 2010 14:56:47 -0700 Subject: [PATCH] gsm: fix argument validation in SetAllowedMode --- src/mm-modem-gsm-network.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/mm-modem-gsm-network.c b/src/mm-modem-gsm-network.c index 7f7b655b..4cd69218 100644 --- a/src/mm-modem-gsm-network.c +++ b/src/mm-modem-gsm-network.c @@ -540,11 +540,11 @@ impl_gsm_modem_set_allowed_mode (MMModemGsmNetwork *modem, MMModemGsmAllowedMode mode, DBusGMethodInvocation *context) { - if (!check_for_single_value (mode)) { + if (mode > MM_MODEM_GSM_ALLOWED_MODE_LAST) { GError *error; - error = g_error_new_literal (MM_MODEM_ERROR, MM_MODEM_ERROR_OPERATION_NOT_SUPPORTED, - "Invalid arguments (more than one value given)"); + error = g_error_new (MM_MODEM_ERROR, MM_MODEM_ERROR_OPERATION_NOT_SUPPORTED, + "Unknown allowed mode %d", mode); dbus_g_method_return_error (context, error); g_error_free (error); return;