Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

#2055 MultiFrequencyTunerChannelSource Null Pointer Error On Channel … #2070

Merged
merged 1 commit into from
Nov 1, 2024

Conversation

DSheirer
Copy link
Owner

@DSheirer DSheirer commented Nov 1, 2024

#2055 Rework the MultiFrequencyTunerChannelSource Null Pointer Error that was happening on startup where the channel rotation monitor was causing the channel to rotate to the next frequency and invoke stop on the current channel source before the channel had started, on some systems where the startup sequence takes a bit longer than normal.

Different solution to resolve possible memory leak with previous solution.

Also resolves issue introduced with #2068 with P25 traffic channel manager where a zero frequency value channel doesn't get started and the traffic channel wasn't being returned to the queue. This normally only impacts on startup when the P25 frequency bands hadn't yet been received.

@DSheirer DSheirer added this to the Version 0.6.1 milestone Nov 1, 2024
@DSheirer DSheirer self-assigned this Nov 1, 2024
…Source Shutdown - different solution to resolve possible memory leak with previous solution. Also fixes P25 issue where a traffic channel that doesn't get started is returned to the traffic channel pool.
@DSheirer DSheirer force-pushed the 2055-null-polyphase-channelizer-error branch from fad1ea3 to 6edab7d Compare November 1, 2024 09:26
@DSheirer DSheirer merged commit df0d046 into master Nov 1, 2024
3 checks passed
@DSheirer DSheirer deleted the 2055-null-polyphase-channelizer-error branch November 1, 2024 09:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant