Skip to content

Commit

Permalink
2023-06-21 Fred Gleason <[email protected]>
Browse files Browse the repository at this point in the history
	* Fixed a regression in rdcatch(1) that caused a phantom event
	entry to appear in the Event List while processing the
	'Start Record Deck' ['RS'] RML.

Signed-off-by: Fred Gleason <[email protected]>
  • Loading branch information
ElvishArtisan committed Jun 21, 2023
1 parent f36fdc5 commit b6c49a5
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 1 deletion.
4 changes: 4 additions & 0 deletions ChangeLog
Original file line number Diff line number Diff line change
Expand Up @@ -24248,3 +24248,7 @@
2023-06-21 Fred Gleason <[email protected]>
* Fixed a bug in rdcatchd(8) that would cause a segfault after
reception of a 'Start Record Deck' ['RS'] RML.
2023-06-21 Fred Gleason <[email protected]>
* Fixed a regression in rdcatch(1) that caused a phantom event
entry to appear in the Event List while processing the
'Start Record Deck' ['RS'] RML.
5 changes: 4 additions & 1 deletion rdcatch/recordlistmodel.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -319,7 +319,6 @@ void RecordListModel::setRecordStatus(const QModelIndex &row,
void RecordListModel::setRecordStatus(unsigned rec_id,RDDeck::Status status)
{
int index=d_ids.indexOf(rec_id);

if(index>=0) {
if(d_statuses.at(index)!=status) {
d_statuses[index]=status;
Expand Down Expand Up @@ -364,6 +363,10 @@ void RecordListModel::channelCounts(int chan,int *waiting,int *active,

QModelIndex RecordListModel::addRecord(unsigned id)
{
if(id>=RDCATCHD_DYNAMIC_BASE_ID) {
return QModelIndex(); // Don't process dynamic record events!
}

//
// Find the insertion offset
//
Expand Down

0 comments on commit b6c49a5

Please sign in to comment.