[Asterisk-code-review] main/utils: allow checking for command in $PATH (asterisk[master])

Sean Bright asteriskteam at digium.com
Wed Jul 27 14:34:04 CDT 2022


Attention is currently required from: Joshua Colp, Philip Prindeville.
Sean Bright has posted comments on this change. ( https://gerrit.asterisk.org/c/asterisk/+/18839 )

Change subject: main/utils: allow checking for command in $PATH
......................................................................


Patch Set 2: Code-Review-1

(1 comment)

File main/utils.c:

https://gerrit.asterisk.org/c/asterisk/+/18839/comment/b5113328_cbabbe1d 
PS2, Line 3236: 		snprintf(filename, sizeof(filename), "%s/%s", token, cmd);
Check for truncation



-- 
To view, visit https://gerrit.asterisk.org/c/asterisk/+/18839
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Change-Id: I4b6f7264c8c737c476c798d2352f3232b263bbdf
Gerrit-Change-Number: 18839
Gerrit-PatchSet: 2
Gerrit-Owner: Philip Prindeville <philipp at redfish-solutions.com>
Gerrit-Reviewer: Benjamin Keith Ford <bford at digium.com>
Gerrit-Reviewer: Friendly Automation
Gerrit-Reviewer: Joshua Colp <jcolp at sangoma.com>
Gerrit-Reviewer: Sean Bright <sean at seanbright.com>
Gerrit-Attention: Joshua Colp <jcolp at sangoma.com>
Gerrit-Attention: Philip Prindeville <philipp at redfish-solutions.com>
Gerrit-Comment-Date: Wed, 27 Jul 2022 19:34:04 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20220727/732e938e/attachment.html>


More information about the asterisk-code-review mailing list