diff options
author | Brian Gix <brian.gix@intel.com> | 2022-06-29 14:16:40 -0700 |
---|---|---|
committer | Luiz Augusto von Dentz <luiz.von.dentz@intel.com> | 2022-06-30 13:27:21 -0700 |
commit | 31690310c096994c553c373ab5464cc39fbfe860 (patch) | |
tree | 4c8dbf5f8ec711d6b37ce56e160a526bcaec921b /src/device.c | |
parent | 8fc3368db84035bee91ce0bea2f7592343e19f81 (diff) | |
download | bluez-31690310c096994c553c373ab5464cc39fbfe860.tar.gz |
core: Fix signed vs unsigned compare
__time_t is not a portable data type, and can cause sign mismatch on
some compares.
Fixes:
CC src/bluetoothd-device.o
src/device.c: In function ‘device_is_name_resolve_allowed’:
src/device.c:4092:17: error: comparison of integer expressions of
different signedness: ‘__time_t’ {aka ‘long int’} and
‘long unsigned int’ [-Werror=sign-compare]
if (now.tv_sec >= device->name_resolve_failed_time +
^~
cc1: all warnings being treated as errors
Diffstat (limited to 'src/device.c')
-rw-r--r-- | src/device.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/device.c b/src/device.c index c5484ca7b..9a4f2ce45 100644 --- a/src/device.c +++ b/src/device.c @@ -4089,8 +4089,8 @@ bool device_is_name_resolve_allowed(struct btd_device *device) /* now >= failed_time + name_request_retry_delay, meaning the * period of not sending name request is over. */ - if (now.tv_sec >= device->name_resolve_failed_time + - btd_opts.name_request_retry_delay) + if (now.tv_sec >= (time_t)(device->name_resolve_failed_time + + btd_opts.name_request_retry_delay)) return true; return false; |