RE: monitor: clamp max_len in print_packet
| Newsgroups | org.kernel.vger.linux-bluetooth |
|---|---|
| Message-ID | <[email protected]> |
This is automated email and please do not reply to this email!
Dear submitter,
Thank you for submitting the patches to the linux bluetooth mailing list.
This is a CI test results with your patch series:
PW Link:https://patchwork.kernel.org/project/bluetooth/list/?series=1146334
---Test result---
Test Summary:
CheckPatch PASS 0.49 seconds
GitLint FAIL 0.34 seconds
BuildEll PASS 20.71 seconds
BluezMake PASS 583.87 seconds
MakeCheck PASS 0.98 seconds
MakeDistcheck PASS 156.41 seconds
CheckValgrind PASS 150.41 seconds
CheckSmatch WARNING 298.13 seconds
bluezmakeextell PASS 96.55 seconds
IncrementalBuild PASS 563.82 seconds
ScanBuild PASS 905.21 seconds
Details
##############################
Test: GitLint - FAIL
Desc: Run gitlint
Output:
monitor: clamp max_len in print_packet
6: B3 Line contains hard tab characters (\t): " char line[LINE_MAX], ts_str[96], pid_str[140];"
7: B3 Line contains hard tab characters (\t): " int col = num_columns();"
8: B3 Line contains hard tab characters (\t): " ..."
9: B3 Line contains hard tab characters (\t): " int max_len = col - len - extra_len - ts_len - 3;"
11: B3 Line contains hard tab characters (\t): " if (max_len <= 0) {"
12: B3 Line contains hard tab characters (\t): " extra = NULL;"
13: B3 Line contains hard tab characters (\t): " max_len = col - len - ts_len - 3;"
14: B3 Line contains hard tab characters (\t): " }"
16: B3 Line contains hard tab characters (\t): " n = snprintf(line + pos, max_len + 1, "%s%s","
17: B3 Line contains hard tab characters (\t): " label ? ": " : "", text);"
51: B3 Line contains hard tab characters (\t): " cols before after"
52: B3 Line contains hard tab characters (\t): " 10 abort ok"
53: B3 Line contains hard tab characters (\t): " 20 abort ok"
54: B3 Line contains hard tab characters (\t): " 40 abort ok"
55: B3 Line contains hard tab characters (\t): " 80 ok ok"
56: B3 Line contains hard tab characters (\t): " 200 ok ok"
57: B3 Line contains hard tab characters (\t): " 2000 ok ok"
58: B3 Line contains hard tab characters (\t): " 3000 abort ok"
59: B3 Line contains hard tab characters (\t): " 5000 abort ok"
##############################
Test: CheckSmatch - WARNING
Desc: Run smatch tool with source
Output:
monitor/packet.c:2015:26: warning: Variable length array is used.monitor/packet.c: note: in included file:monitor/bt.h:3924:52: warning: array of flexible structuresmonitor/bt.h:3912:40: warning: array of flexible structures
https://github.com/bluez/bluez/pull/2404
---
Regards,
Linux Bluetooth