| Message ID | 20240702073955.10962-1-r.smirnov@omp.ru |
|---|---|
| State | New |
| Headers | show |
| Series | [BlueZ,v1] audio/a2dp: add NULL checks to find_remote_sep() | expand |
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=867428 ---Test result--- Test Summary: CheckPatch PASS 0.43 seconds GitLint PASS 0.30 seconds BuildEll PASS 26.44 seconds BluezMake PASS 1611.40 seconds MakeCheck PASS 13.31 seconds MakeDistcheck PASS 176.99 seconds CheckValgrind PASS 250.12 seconds CheckSmatch PASS 351.41 seconds bluezmakeextell PASS 125.11 seconds IncrementalBuild PASS 1551.65 seconds ScanBuild PASS 984.95 seconds --- Regards, Linux Bluetooth
Hello: This patch was applied to bluetooth/bluez.git (master) by Luiz Augusto von Dentz <luiz.von.dentz@intel.com>: On Tue, 2 Jul 2024 10:39:55 +0300 you wrote: > Make find_remote_sep() safe for passing NULL pointers. > > Found with the SVACE static analysis tool. > --- > profiles/audio/a2dp.c | 3 +++ > 1 file changed, 3 insertions(+) Here is the summary with links: - [BlueZ,v1] audio/a2dp: add NULL checks to find_remote_sep() https://git.kernel.org/pub/scm/bluetooth/bluez.git/?id=922a8a8bd494 You are awesome, thank you!
diff --git a/profiles/audio/a2dp.c b/profiles/audio/a2dp.c index a3c294bc3..43da38051 100644 --- a/profiles/audio/a2dp.c +++ b/profiles/audio/a2dp.c @@ -1327,6 +1327,9 @@ static struct a2dp_remote_sep *find_remote_sep(struct a2dp_channel *chan, { struct avdtp_remote_sep *rsep; + if (!chan || !sep) + return NULL; + rsep = avdtp_find_remote_sep(chan->session, sep->lsep); return queue_find(chan->seps, match_remote_sep, rsep);