mirror of
https://git.kernel.org/pub/scm/bluetooth/bluez.git
synced 2024-11-25 13:14:14 +08:00
hcitool: Fix errno handling convention
Variables which are assigned to the errno variable (usually called "err") should be negative, and "-err" should be used where a positive value is needed.
This commit is contained in:
parent
7f7d66842b
commit
100527bd5a
@ -2696,9 +2696,9 @@ static void cmd_lewladd(int dev_id, int argc, char **argv)
|
||||
hci_close_dev(dd);
|
||||
|
||||
if (err < 0) {
|
||||
err = errno;
|
||||
err = -errno;
|
||||
fprintf(stderr, "Can't add to white list: %s(%d)\n",
|
||||
strerror(err), err);
|
||||
strerror(-err), -err);
|
||||
exit(1);
|
||||
}
|
||||
}
|
||||
@ -2786,9 +2786,9 @@ static void cmd_lewlsz(int dev_id, int argc, char **argv)
|
||||
hci_close_dev(dd);
|
||||
|
||||
if (err < 0) {
|
||||
err = errno;
|
||||
err = -errno;
|
||||
fprintf(stderr, "Can't read white list size: %s(%d)\n",
|
||||
strerror(err), err);
|
||||
strerror(-err), -err);
|
||||
exit(1);
|
||||
}
|
||||
|
||||
@ -2831,9 +2831,9 @@ static void cmd_lewlclr(int dev_id, int argc, char **argv)
|
||||
hci_close_dev(dd);
|
||||
|
||||
if (err < 0) {
|
||||
err = errno;
|
||||
err = -errno;
|
||||
fprintf(stderr, "Can't clear white list: %s(%d)\n",
|
||||
strerror(err), err);
|
||||
strerror(-err), -err);
|
||||
exit(1);
|
||||
}
|
||||
}
|
||||
@ -2961,9 +2961,9 @@ static void cmd_lecup(int dev_id, int argc, char **argv)
|
||||
|
||||
if (hci_le_conn_update(dd, htobs(handle), htobs(min), htobs(max),
|
||||
htobs(latency), htobs(timeout), 5000) < 0) {
|
||||
int err = errno;
|
||||
int err = -errno;
|
||||
fprintf(stderr, "Could not change connection params: %s(%d)\n",
|
||||
strerror(err), err);
|
||||
strerror(-err), -err);
|
||||
}
|
||||
|
||||
hci_close_dev(dd);
|
||||
|
Loading…
Reference in New Issue
Block a user