Hi, On Mon, Dec 11, 2023 at 6:31 PM <bluez.test.bot@xxxxxxxxx> wrote: > > 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=808938 Could one of you guys please verify that this set doesn't break anything related to broadcast. > ---Test result--- > > Test Summary: > CheckPatch PASS 2.70 seconds > GitLint FAIL 1.73 seconds > BuildEll PASS 24.71 seconds > BluezMake PASS 780.94 seconds > MakeCheck PASS 12.60 seconds > MakeDistcheck PASS 164.82 seconds > CheckValgrind PASS 225.11 seconds > CheckSmatch PASS 330.30 seconds > bluezmakeextell PASS 103.81 seconds > IncrementalBuild PASS 4181.08 seconds > ScanBuild WARNING 990.81 seconds > > Details > ############################## > Test: GitLint - FAIL > Desc: Run gitlint > Output: > [BlueZ,v4,2/6] shared/bap: Make bt_bap_select match the channel map > > WARNING: I3 - ignore-body-lines: gitlint will be switching from using Python regex 'match' (match beginning) to 'search' (match anywhere) semantics. Please review your ignore-body-lines.regex option accordingly. To remove this warning, set general.regex-style-search=True. More details: https://jorisroovers.github.io/gitlint/configuration/#regex-style-search > 13: B3 Line contains hard tab characters (\t): " 0000a0201030202010304280001020206000000000a020103020201030428" > 14: B3 Line contains hard tab characters (\t): " 0002020206000000000a02010302020103042800" > [BlueZ,v4,3/6] org.bluez.MediaEndpoint: Add ChannelAllocation to SelectProperties > > WARNING: I3 - ignore-body-lines: gitlint will be switching from using Python regex 'match' (match beginning) to 'search' (match anywhere) semantics. Please review your ignore-body-lines.regex option accordingly. To remove this warning, set general.regex-style-search=True. More details: https://jorisroovers.github.io/gitlint/configuration/#regex-style-search > 1: T1 Title exceeds max length (81>80): "[BlueZ,v4,3/6] org.bluez.MediaEndpoint: Add ChannelAllocation to SelectProperties" > [BlueZ,v4,6/6] client/player: Use ChannelAllocation given on SelectProperties > > WARNING: I3 - ignore-body-lines: gitlint will be switching from using Python regex 'match' (match beginning) to 'search' (match anywhere) semantics. Please review your ignore-body-lines.regex option accordingly. To remove this warning, set general.regex-style-search=True. More details: https://jorisroovers.github.io/gitlint/configuration/#regex-style-search > 13: B3 Line contains hard tab characters (\t): " 6020206000000001002010302020103042800050302000000010202060000" > 14: B3 Line contains hard tab characters (\t): " 0000100201030202010304280005030100000002020206000000001002010" > 15: B3 Line contains hard tab characters (\t): " 302020103042800050302000000" > ############################## > Test: ScanBuild - WARNING > Desc: Run Scan Build > Output: > src/shared/bap.c:4766:23: warning: Access to field 'type' results in a dereference of a null pointer (loaded from variable 'lpac') > if (!match.rpac && (lpac->type != BT_BAP_BCAST_SOURCE)) > ^~~~~~~~~~ > 1 warning generated. > In file included from tools/mesh-gatt/crypto.c:32: > ./src/shared/util.h:228:9: warning: 1st function call argument is an uninitialized value > return be32_to_cpu(get_unaligned((const uint32_t *) ptr)); > ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > ./src/shared/util.h:33:26: note: expanded from macro 'be32_to_cpu' > #define be32_to_cpu(val) bswap_32(val) > ^~~~~~~~~~~~~ > /usr/include/byteswap.h:34:21: note: expanded from macro 'bswap_32' > #define bswap_32(x) __bswap_32 (x) > ^~~~~~~~~~~~~~ > In file included from tools/mesh-gatt/crypto.c:32: > ./src/shared/util.h:238:9: warning: 1st function call argument is an uninitialized value > return be64_to_cpu(get_unaligned((const uint64_t *) ptr)); > ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > ./src/shared/util.h:34:26: note: expanded from macro 'be64_to_cpu' > #define be64_to_cpu(val) bswap_64(val) > ^~~~~~~~~~~~~ > /usr/include/byteswap.h:37:21: note: expanded from macro 'bswap_64' > #define bswap_64(x) __bswap_64 (x) > ^~~~~~~~~~~~~~ > 2 warnings generated. > > > > --- > Regards, > Linux Bluetooth > -- Luiz Augusto von Dentz