[Asterisk-code-review] res_rtp_asterisk: More robust timestamp checking (asterisk[16])

Sean Bright asteriskteam at digium.com
Tue Apr 27 15:48:16 CDT 2021


Hello Friendly Automation, 

I'd like you to reexamine a change. Please visit

    https://gerrit.asterisk.org/c/asterisk/+/15807

to look at the new patch set (#2).

Change subject: res_rtp_asterisk: More robust timestamp checking
......................................................................

res_rtp_asterisk: More robust timestamp checking

We assume that a timestamp value of 0 represents an 'uninitialized'
timestamp, but 0 is a valid value. Add a simple wrapper to be able to
differentiate between whether the value is set or not.

This also removes the fix for ASTERISK~28812 which should not be
needed if we are checking the last timestamp appropriately.

ASTERISK-29030 #close

Change-Id: Ie70d657d580d9a1f2877e25a6ef161c5ad761cf7
---
M res/res_rtp_asterisk.c
1 file changed, 19 insertions(+), 9 deletions(-)


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/07/15807/2
-- 
To view, visit https://gerrit.asterisk.org/c/asterisk/+/15807
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: 16
Gerrit-Change-Id: Ie70d657d580d9a1f2877e25a6ef161c5ad761cf7
Gerrit-Change-Number: 15807
Gerrit-PatchSet: 2
Gerrit-Owner: Sean Bright <sean at seanbright.com>
Gerrit-Reviewer: Friendly Automation
Gerrit-MessageType: newpatchset
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20210427/4091aec6/attachment.html>


More information about the asterisk-code-review mailing list