mirror of
https://github.com/curl/curl.git
synced 2026-01-18 17:21:26 +01:00
Replace existing `mk-unity.pl` `--embed` workaround with running
`clang-tidy` manually on individual test source instead. This aligns
with how clang-tidy works and removes `mk-unity.pl` from the solution.
Also:
- mqttd: fix potentially uninitialized buffer by zero filling it.
```
tests/server/mqttd.c:484:41: error: The left operand of '<<' is a garbage value
[clang-analyzer-core.UndefinedBinaryOperatorResult,-warnings-as-errors]
484 | payload_len = (size_t)(buffer[10] << 8) | buffer[11];
| ^
[...]
tests/server/mqttd.c:606:45: error: The left operand of '<<' is a garbage value
[clang-analyzer-core.UndefinedBinaryOperatorResult,-warnings-as-errors]
606 | topiclen = (size_t)(buffer[1 + bytes] << 8) | buffer[2 + bytes];
| ^
```
- sockfilt: fix potential out-of-bound pointer:
```
tests/server/sockfilt.c:1128:33: error: The 2nd argument to 'send' is a buffer
with size 17010 but should be a buffer with size equal to or greater than
the value of the 3rd argument (which is 18446744073709551615)
[clang-analyzer-unix.StdCLibraryFunctions,-warnings-as-errors]
1128 | ssize_t bytes_written = swrite(sockfd, buffer, buffer_len);
| ^
```
- clang-tidy: suppress bogus `bzero()` warnings that happens
inside the notorious `FD_ZERO()` macros, on macOS.
Ref: https://github.com/curl/curl/pull/17680#issuecomment-2991730158
Closes #17705
79 lines
2.2 KiB
Perl
Executable File
79 lines
2.2 KiB
Perl
Executable File
#!/usr/bin/env perl
|
|
#***************************************************************************
|
|
# _ _ ____ _
|
|
# Project ___| | | | _ \| |
|
|
# / __| | | | |_) | |
|
|
# | (__| |_| | _ <| |___
|
|
# \___|\___/|_| \_\_____|
|
|
#
|
|
# Copyright (C) Viktor Szakats
|
|
#
|
|
# This software is licensed as described in the file COPYING, which
|
|
# you should have received as part of this distribution. The terms
|
|
# are also available at https://curl.se/docs/copyright.html.
|
|
#
|
|
# You may opt to use, copy, modify, merge, publish, distribute and/or sell
|
|
# copies of the Software, and permit persons to whom the Software is
|
|
# furnished to do so, under the terms of the COPYING file.
|
|
#
|
|
# This software is distributed on an "AS IS" basis, WITHOUT WARRANTY OF ANY
|
|
# KIND, either express or implied.
|
|
#
|
|
# SPDX-License-Identifier: curl
|
|
#
|
|
###########################################################################
|
|
|
|
# Helper script for "unity"-like support in autotools and to bundle up tests
|
|
# for both autotools and cmake. It generates the umbrella C source that
|
|
# includes the individual source files and tests.
|
|
|
|
use strict;
|
|
use warnings;
|
|
|
|
if(!@ARGV) {
|
|
die "Usage: $0 [--test <tests>] [--include <include-c-sources>]\n";
|
|
}
|
|
|
|
my @src;
|
|
my %include;
|
|
my $in_include = 0;
|
|
my $any_test = 0;
|
|
foreach my $src (@ARGV) {
|
|
if($src eq "--test") {
|
|
$in_include = 0;
|
|
}
|
|
elsif($src eq "--include") {
|
|
$in_include = 1;
|
|
}
|
|
elsif($in_include) {
|
|
$include{$src} = 1;
|
|
push @src, $src;
|
|
}
|
|
else {
|
|
push @src, $src;
|
|
$any_test = 1;
|
|
}
|
|
}
|
|
|
|
print "/* !checksrc! disable COPYRIGHT all */\n\n";
|
|
if($any_test) {
|
|
print "#include \"first.h\"\n\n";
|
|
}
|
|
|
|
my $tlist = "";
|
|
|
|
foreach my $src (@src) {
|
|
if($src =~ /([a-z0-9_]+)\.c$/) {
|
|
my $name = $1;
|
|
print "#include \"$src\"\n";
|
|
if(not exists $include{$src}) { # register test entry function
|
|
$tlist .= " {\"$name\", test_$name},\n";
|
|
}
|
|
}
|
|
}
|
|
|
|
if($any_test) {
|
|
print "\nconst struct entry_s s_entries[] = {\n$tlist {NULL, NULL}\n};\n";
|
|
print "\n#include \"first.c\"\n";
|
|
}
|