diff options
| author | Nick Pelly <npelly@google.com> | 2009-09-18 11:37:06 -0700 |
|---|---|---|
| committer | Nick Pelly <npelly@google.com> | 2009-09-18 11:37:06 -0700 |
| commit | 6ef935c90c0988645d33529f4ddbe058977c765c (patch) | |
| tree | 9b348c14964da373530f8f5533164c1809f89d5a /framework/java/android/bluetooth/BluetoothAdapter.java | |
| parent | 51eae29605b1be4b089eed344784a46cb9895f67 (diff) | |
Reject lowercase characters in checkBluetoothAddress().
This keeps consistency with Bluez which uses upper case string address. It's
important to keep the case the same so that .equals() in BluetoothService.java
work.
Change-Id: I6404ca137d0aec3cc2e6e7cb79763d5305a03547
Diffstat (limited to 'framework/java/android/bluetooth/BluetoothAdapter.java')
| -rw-r--r-- | framework/java/android/bluetooth/BluetoothAdapter.java | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/framework/java/android/bluetooth/BluetoothAdapter.java b/framework/java/android/bluetooth/BluetoothAdapter.java index 96a927b30c..1fc22fe524 100644 --- a/framework/java/android/bluetooth/BluetoothAdapter.java +++ b/framework/java/android/bluetooth/BluetoothAdapter.java @@ -573,6 +573,7 @@ public final class BluetoothAdapter { /** * Validate a Bluetooth address, such as "00:43:A8:23:10:F0" + * <p>Alphabetic characters must be uppercase to be valid. * * @param address Bluetooth address as string * @return true if the address is valid, false otherwise @@ -586,8 +587,9 @@ public final class BluetoothAdapter { switch (i % 3) { case 0: case 1: - if (Character.digit(c, 16) != -1) { - break; // hex character, OK + if ((c >= '0' && c <= '9') || (c >= 'A' && c <= 'F')) { + // hex character, OK + break; } return false; case 2: |