Received-SPF: pass (google.com: domain of linux-bluetooth-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <630e99f1.620a0220.31c2d.a8b4@mx.google.com>
Date:   Tue, 30 Aug 2022 16:14:57 -0700 (PDT)
Content-Type: multipart/mixed; boundary="===============2249218721927588961=="
MIME-Version: 1.0
From:   bluez.test.bot@gmail.com
To:     linux-bluetooth@vger.kernel.org, luiz.dentz@gmail.com
Subject: RE: [BlueZ,1/2] client/player: Fix checkpatch warning
Reply-To: linux-bluetooth@vger.kernel.org
In-Reply-To: <20220830214215.1137276-1-luiz.dentz@gmail.com>
References: <20220830214215.1137276-1-luiz.dentz@gmail.com>
Precedence: bulk

--===============2249218721927588961==
Content-Type: text/plain; charset="us-ascii"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit

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=672614

---Test result---

Test Summary:
CheckPatch                    FAIL      1.39 seconds
GitLint                       PASS      0.96 seconds
Prep - Setup ELL              PASS      32.15 seconds
Build - Prep                  PASS      0.65 seconds
Build - Configure             PASS      10.33 seconds
Build - Make                  PASS      980.25 seconds
Make Check                    PASS      12.90 seconds
Make Check w/Valgrind         PASS      347.96 seconds
Make Distcheck                PASS      300.51 seconds
Build w/ext ELL - Configure   PASS      10.48 seconds
Build w/ext ELL - Make        PASS      101.00 seconds
Incremental Build w/ patches  PASS      242.98 seconds
Scan Build                    WARNING   684.12 seconds

Details
##############################
Test: CheckPatch - FAIL
Desc: Run checkpatch.pl script with rule in .checkpatch.conf
Output:
[BlueZ,1/2] client/player: Fix checkpatch warning
WARNING:EMAIL_SUBJECT: A patch subject line should describe the change not the tool that found it
#69: 
Subject: [PATCH BlueZ 1/2] client/player: Fix checkpatch warning

/github/workspace/src/12959894.patch total: 0 errors, 1 warnings, 24 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.

/github/workspace/src/12959894.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: Scan Build - WARNING
Desc: Run Scan Build with patches
Output:
*****************************************************************************
The bugs reported by the scan-build may or may not be caused by your patches.
Please check the list and fix the bugs if they are caused by your patch.
*****************************************************************************
client/player.c:1755:25: warning: Dereference of null pointer
        iov_append(&cfg->caps, preset->data.iov_base, preset->data.iov_len);
                               ^~~~~~~~~~~~~~~~~~~~~
1 warning generated.
src/shared/shell.c:1136:19: warning: Null pointer passed to 1st parameter expecting 'nonnull'
                        data.timeout = strtol(optarg, &endptr, 0);
                                       ^~~~~~~~~~~~~~~~~~~~~~~~~~
src/shared/shell.c:1158:13: warning: Access to field 'options' results in a dereference of a null pointer (loaded from variable 'opt')
                        if (c != opt->options[index - offset].val) {
                                 ^~~~~~~~~~~~
2 warnings generated.


---
Regards,
Linux Bluetooth


--===============2249218721927588961==--