[Asterisk-code-review] taskprocessor.c: Prevent crash on graceful shutdown (asterisk[16])

[Michael Bradeen]

Attention is currently required from: Joshua Colp, Benjamin Keith Ford.
Michael Bradeen has posted comments on this change. ( https://gerrit.asterisk.org/c/asterisk/+/17955 )

Change subject: taskprocessor.c: Prevent crash on graceful shutdown
......................................................................


Patch Set 4:

(2 comments)

File main/taskprocessor.c:

https://gerrit.asterisk.org/c/asterisk/+/17955/comment/8fe5c4c6_b3b869b7 
PS3, Line 293: 	int objcount, tries;
> Very minor, so feel free to ignore, but I would put these on 2 separate lines in case something need […]
Done


https://gerrit.asterisk.org/c/asterisk/+/17955/comment/c41966ce_93b70e24 
PS3, Line 307: 		/* give the running taskprocessors a chance to finish, up to 5 seconds */
> In taskprocessor.h, you've got 10 instead of 5. […]
Done



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

Gerrit-Project: asterisk
Gerrit-Branch: 16
Gerrit-Change-Id: Ia932fc003d316389b9c4fd15ad6594458c9727f1
Gerrit-Change-Number: 17955
Gerrit-PatchSet: 4
Gerrit-Owner: Michael Bradeen <mbradeen at sangoma.com>
Gerrit-Reviewer: Benjamin Keith Ford <bford at digium.com>
Gerrit-Reviewer: Friendly Automation
Gerrit-Reviewer: Joshua Colp <jcolp at sangoma.com>
Gerrit-Attention: Joshua Colp <jcolp at sangoma.com>
Gerrit-Attention: Benjamin Keith Ford <bford at digium.com>
Gerrit-Comment-Date: Fri, 04 Feb 2022 17:14:42 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Benjamin Keith Ford <bford at digium.com>
Gerrit-MessageType: comment
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20220204/be319a2d/attachment-0001.html>