nimbleape / asterisk-dialogflow-rtp-audioserver

MIT License
38 stars 18 forks source link

potential timestamp issue #2

Open danjenkins opened 4 years ago

danjenkins commented 4 years ago

https://nodejs.org/api/buffer.html#buffer_buf_writeuint32be_value_offset Buffer | Node.js v13.10.1 Documentation Node.js About these Docs Usage & Example Assertion Testing Async Hooks Buffer C++ Addons C/C++ Addons with N-API Child Processes Cluster Command Line Options so if any of your fields go out of range, behavior is undefined. so you should make sure to handle that explicitly. probably more of a issue for your sequence number and not necessarily the timestamp since the sequence number is only 16 bit