diff options
author | Gleb Smirnoff <glebius@FreeBSD.org> | 2024-03-11 14:50:29 +0000 |
---|---|---|
committer | Gleb Smirnoff <glebius@FreeBSD.org> | 2024-03-11 14:50:29 +0000 |
commit | 55951611e49a5a43ba151582505019c38558c0e7 (patch) | |
tree | 54c3f3b70b951eb7d77a9879ac115526df8d192c /tests | |
parent | 59c8e88e72633afbc47a4ace0d2170d00d51f7dc (diff) | |
download | src-55951611e49a5a43ba151582505019c38558c0e7.tar.gz src-55951611e49a5a43ba151582505019c38558c0e7.zip |
tests/unix_seqpacket: mk_pair_of_sockets() requires a valid argument
All callers do it right. Don't be overprotective against a stupid
caller and thus don't look like a code that leaks a resource.
Reported by: Coverity Scan
CID: 1539210
Diffstat (limited to 'tests')
-rw-r--r-- | tests/sys/kern/unix_seqpacket_test.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/tests/sys/kern/unix_seqpacket_test.c b/tests/sys/kern/unix_seqpacket_test.c index e93bdfe7e0b1..ca3fcdef9e6a 100644 --- a/tests/sys/kern/unix_seqpacket_test.c +++ b/tests/sys/kern/unix_seqpacket_test.c @@ -124,10 +124,8 @@ mk_pair_of_sockets(int *sv) atf_tc_fail("accept(2) failed"); } - if (sv != NULL) { - sv[0] = s1; - sv[1] = s2; - } + sv[0] = s1; + sv[1] = s2; close(s); |