summaryrefslogtreecommitdiffstats
path: root/src/common.c
diff options
context:
space:
mode:
authorMarcel Holtmann <marcel@holtmann.org>2010-03-20 02:26:57 +0100
committerMarcel Holtmann <marcel@holtmann.org>2010-03-20 02:26:57 +0100
commit256c6e14c7947a10833cfcacf2ee74422d0d7563 (patch)
treec38f73fa317a9c4d3dfe4af300f79cf573273366 /src/common.c
parentd6e78f78287599c94dde744e1b856b27f6d1b884 (diff)
downloadofono-256c6e14c7947a10833cfcacf2ee74422d0d7563.tar.bz2
Revert the default statement for PIN type checking
Also fix the coding style for the switch statement and remove the number assignment for an internal enum.
Diffstat (limited to 'src/common.c')
-rw-r--r--src/common.c5
1 files changed, 1 insertions, 4 deletions
diff --git a/src/common.c b/src/common.c
index 32b64d49..b58d9bee 100644
--- a/src/common.c
+++ b/src/common.c
@@ -592,8 +592,7 @@ gboolean is_valid_pin(const char *pin, enum pin_type type)
if (i != strspn(pin, "012345679"))
return FALSE;
- switch (type)
- {
+ switch (type) {
case PIN_TYPE_PIN:
/* 11.11 Section 9.3 ("CHV"): 4..8 IA-5 digits */
if (4 <= i && i <= 8)
@@ -613,8 +612,6 @@ gboolean is_valid_pin(const char *pin, enum pin_type type)
if (i < 8)
return TRUE;
break;
- default:
- break;
}
return FALSE;