BUGFIX: null reference when trying to perform old style lookup of timestamp when debugging is enabled;

gatekeep · gatekeep · commit 236073a2d379 · 2025-09-19T12:39:12.000-04:00
diff --git a/src/common/network/FrameQueue.cpp b/src/common/network/FrameQueue.cpp
@@ -301,9 +301,10 @@ uint8_t* FrameQueue::generateMessage(const uint8_t* message, uint32_t length, ui
         }
 
         if (timestamp != INVALID_TS) {
+            uint32_t prevTimestamp = timestamp;
             timestamp += (RTP_GENERIC_CLOCK_RATE / 133);
             if (m_debug)
-                LogDebugEx(LOG_NET, "FrameQueue::generateMessage()", "RTP streamId = %u, previous TS = %u, TS = %u, rtpSeq = %u", streamId, m_streamTimestamps[streamId], timestamp, rtpSeq);
+                LogDebugEx(LOG_NET, "FrameQueue::generateMessage()", "RTP streamId = %u, previous TS = %u, TS = %u, rtpSeq = %u", streamId, prevTimestamp, timestamp, rtpSeq);
             updateTimestamp(streamId, timestamp);
         }
     }

Original file line number	Diff line number	Diff line change
`@@ -301,9 +301,10 @@ uint8_t* FrameQueue::generateMessage(const uint8_t* message, uint32_t length, ui`
`301`	`301`	`}`
`302`	`302`
`303`	`303`	`if (timestamp != INVALID_TS) {`
	`304`	`+ uint32_t prevTimestamp = timestamp;`
`304`	`305`	`timestamp += (RTP_GENERIC_CLOCK_RATE / 133);`
`305`	`306`	`if (m_debug)`
`306`		`- LogDebugEx(LOG_NET, "FrameQueue::generateMessage()", "RTP streamId = %u, previous TS = %u, TS = %u, rtpSeq = %u", streamId, m_streamTimestamps[streamId], timestamp, rtpSeq);`
	`307`	`+ LogDebugEx(LOG_NET, "FrameQueue::generateMessage()", "RTP streamId = %u, previous TS = %u, TS = %u, rtpSeq = %u", streamId, prevTimestamp, timestamp, rtpSeq);`
`307`	`308`	`updateTimestamp(streamId, timestamp);`
`308`	`309`	`}`
`309`	`310`	`}`