diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2014-11-04 03:55:09 -0500 |
|---|---|---|
| committer | Marcel Holtmann <marcel@holtmann.org> | 2014-11-04 11:44:48 -0500 |
| commit | 0d8a52f933f817d0b62955a5a362fb7f2508f06c (patch) | |
| tree | b886a45331b0bd09cf4a74d5f9a43e7448a05043 /include/linux | |
| parent | 980edbd503dc0823c75e138edc53a48f99e1e3f6 (diff) | |
ieee802154: || vs && in ieee802154_is_valid_extended_addr()
The ieee802154_is_valid_extended_addr() always returns true because
there is a typo. The || should be &&. Neither 0x0000000000000000ULL
nor 0xffffffffffffffffULL are valid addresses.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Acked-by: Alexander Aring <alex.aring@gmail.com>
Signed-off-by: Marcel Holtmann <marcel@holtmann.org>
Diffstat (limited to 'include/linux')
| -rw-r--r-- | include/linux/ieee802154.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/linux/ieee802154.h b/include/linux/ieee802154.h index 5d9e7459d94b..4c032863cd71 100644 --- a/include/linux/ieee802154.h +++ b/include/linux/ieee802154.h | |||
| @@ -209,7 +209,7 @@ static inline bool ieee802154_is_valid_extended_addr(const __le64 addr) | |||
| 209 | * This is currently a workaround because neighbor discovery can't | 209 | * This is currently a workaround because neighbor discovery can't |
| 210 | * deal with short addresses types right now. | 210 | * deal with short addresses types right now. |
| 211 | */ | 211 | */ |
| 212 | return ((addr != cpu_to_le64(0x0000000000000000ULL)) || | 212 | return ((addr != cpu_to_le64(0x0000000000000000ULL)) && |
| 213 | (addr != cpu_to_le64(0xffffffffffffffffULL))); | 213 | (addr != cpu_to_le64(0xffffffffffffffffULL))); |
| 214 | } | 214 | } |
| 215 | 215 | ||
