| Message ID | 20210916122442.14732-1-wangxiang@cdjrlc.com (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | 98dc68f8b0c2248bdc3688c90d2499247b9432e4 |
| Headers | show |
| Series | [v3] selftests: nci: replace unsigned int with int | expand |
Hello: This patch was applied to netdev/net.git (refs/heads/master): On Thu, 16 Sep 2021 20:24:42 +0800 you wrote: > Should not use comparison of unsigned expressions < 0. > > Signed-off-by: Xiang wangx <wangxiang@cdjrlc.com> > --- > > Changes since v1 > * Change commit log > > [...] Here is the summary with links: - [v3] selftests: nci: replace unsigned int with int https://git.kernel.org/netdev/net/c/98dc68f8b0c2 You are awesome, thank you! -- Deet-doot-dot, I am a bot. https://korg.docs.kernel.org/patchwork/pwbot.html
diff --git a/tools/testing/selftests/nci/nci_dev.c b/tools/testing/selftests/nci/nci_dev.c index e1bf55dabdf6..162c41e9bcae 100644 --- a/tools/testing/selftests/nci/nci_dev.c +++ b/tools/testing/selftests/nci/nci_dev.c @@ -746,7 +746,7 @@ int read_write_nci_cmd(int nfc_sock, int virtual_fd, const __u8 *cmd, __u32 cmd_ const __u8 *rsp, __u32 rsp_len) { char buf[256]; - unsigned int len; + int len; send(nfc_sock, &cmd[3], cmd_len - 3, 0); len = read(virtual_fd, buf, cmd_len);
Should not use comparison of unsigned expressions < 0. Signed-off-by: Xiang wangx <wangxiang@cdjrlc.com> --- Changes since v1 * Change commit log Changes since v2 * Change commit log tools/testing/selftests/nci/nci_dev.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)