RE: [bluez] monitor: Fix stack-buffer-overflow in packet_hci_scodata()

[bluez.test.bot@gmail.com]

[-- Attachment #1: Type: text/plain, Size: 4487 bytes --]

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/series/1173024/

---Test result---

Test Summary:
CheckPatch                    FAIL      0.53 seconds
GitLint                       FAIL      0.33 seconds
BuildEll                      PASS      15.45 seconds
BluezMake                     PASS      285.73 seconds
MakeCheck                     PASS      0.76 seconds
MakeDistcheck                 PASS      117.73 seconds
CheckValgrind                 PASS      110.23 seconds
CheckSmatch                   WARNING   216.94 seconds
bluezmakeextell               PASS      71.69 seconds
TestFunctional                PASS      891.85 seconds
IncrementalBuild              PASS      279.95 seconds
ScanBuild                     PASS      794.28 seconds

Details
##############################
Test: CheckPatch - FAIL
Desc: Run checkpatch.pl script
Output:
[bluez] monitor: Fix stack-buffer-overflow in packet_hci_scodata()
WARNING:COMMIT_LOG_LONG_LINE: Prefer a maximum 75 chars per line (possible unwrapped commit description?)
#163: 
AddressSanitizer: stack-buffer-overflow on address 0x7bd1443dedba at pc 0x7bd1474ced69 bp 0x7ffdb0324880 sp 0x7ffdb0324010

WARNING:LONG_LINE: line length of 97 exceeds 80 columns
#266: FILE: monitor/packet.c:15055:
+		snprintf(handle_str, sizeof(handle_str), "Handle %d [%u/%u]", acl_handle(handle),

WARNING:LONG_LINE: line length of 90 exceeds 80 columns
#270: FILE: monitor/packet.c:15058:
+		snprintf(handle_str, sizeof(handle_str), "Handle %d", acl_handle(handle));

WARNING:LONG_LINE: line length of 97 exceeds 80 columns
#324: FILE: monitor/packet.c:15129:
+		snprintf(handle_str, sizeof(handle_str), "Handle %d [%u/%u]", acl_handle(handle),

WARNING:LONG_LINE: line length of 90 exceeds 80 columns
#328: FILE: monitor/packet.c:15132:
+		snprintf(handle_str, sizeof(handle_str), "Handle %d", acl_handle(handle));

WARNING:LONG_LINE: line length of 100 exceeds 80 columns
#354: FILE: monitor/packet.c:15242:
+		snprintf(handle_str, sizeof(handle_str), "Handle %u%s", acl_handle(handle), sn_str);

/home/runner/work/bluez/bluez/src/patch/14844485.patch total: 0 errors, 6 warnings, 164 lines checked

NOTE: For some of the reported defects, checkpatch may be able to
      mechanically convert to the typical style using --fix or --fix-inplace.

/home/runner/work/bluez/bluez/src/patch/14844485.patch has style problems, please review.

NOTE: Ignored message types: COMMIT_MESSAGE COMPLEX_MACRO CONST_STRUCT FILE_PATH_CHANGES MISSING_SIGN_OFF PREFER_PACKED SPDX_LICENSE_TAG SPLIT_STRING SSCANF_TO_KSTRTO

NOTE: If any of the errors are false positives, please report
      them to the maintainer, see CHECKPATCH in MAINTAINERS.


##############################
Test: GitLint - FAIL
Desc: Run gitlint
Output:
[bluez] monitor: Fix stack-buffer-overflow in packet_hci_scodata()

8: B1 Line exceeds max length (122>80): "AddressSanitizer: stack-buffer-overflow on address 0x7bd1443dedba at pc 0x7bd1474ced69 bp 0x7ffdb0324880 sp 0x7ffdb0324010"
10: B1 Line exceeds max length (118>80): "    #0 0x7bd1474ced68 in vsprintf ../../../../src/libsanitizer/sanitizer_common/sanitizer_common_interceptors.inc:1671"
11: B1 Line exceeds max length (123>80): "    #1 0x7bd1474d0643 in __sprintf_chk ../../../../src/libsanitizer/sanitizer_common/sanitizer_common_interceptors.inc:1719"
18: B1 Line exceeds max length (86>80): "    #8 0x7bd1474e81d86 in mainloop_run_with_signal ../src/shared/mainloop-notify.c:196"
20: B1 Line exceeds max length (90>80): "    #10 0x7bd14662a1c9 in __libc_start_call_main ../sysdeps/nptl/libc_start_call_main.h:58"
21: B1 Line exceeds max length (81>80): "    #11 0x7bd14662a28a in __libc_start_main_impl ../sysdeps/nptl/libc-start.c:360"
22: B1 Line exceeds max length (110>80): "    #12 0x5f5c39cbc664 in _start (/usr/bin/btmon+0x2b2664) (BuildId: 370a6386c9fceae18f9a456c0c029ac378b0b070)"
34: B2 Line has trailing whitespace: "--  "
##############################
Test: CheckSmatch - WARNING
Desc: Run smatch tool with source
Output:
monitor/packet.c:2289: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/2576

---
Regards,
Linux Bluetooth