Ticket #9748 (closed: fixed)

Opened 6 years ago

Last modified 5 years ago

Another race condition in the SNS live listener

Reported by: Ross Miller Owned by: Ross Miller
Priority: major Milestone: Release 3.2
Component: Framework Keywords:
Cc: Blocked By:
Blocking: Tester: Vickie Lynch

Description

Changes introduced by ticket #9397 mean we need to lock a mutex in the runStatus() function.

Change History

comment:1 Changed 6 years ago by Nick Draper

  • Status changed from new to assigned

comment:2 Changed 6 years ago by Ross Miller

  • Status changed from assigned to inprogress

comment:3 Changed 6 years ago by Ross Miller

Lock mutex in call to runStatus()

Lock the mutex near the top of the runStatus function. Until recently, this wasn't necessary, but #9397 made it possible for the background thread to change m_status and m_deferredRunDetailsPkt while runStatus is executing in the foreground.

Refs #9748

Changeset: 5358ec770da4f98b2c8fe9443d925adc795a6bd8

comment:4 Changed 6 years ago by Ross Miller

Minor comment & log updates

Fixed a few comments and also changed a few log statements from debug level to warning level.

Refs #9748

Changeset: d07e0041017f554cb0f646e6579a10fb4752a23d

comment:5 Changed 6 years ago by Ross Miller

  • Status changed from inprogress to verify
  • Resolution set to fixed

comment:6 Changed 6 years ago by Vickie Lynch

  • Status changed from verify to verifying
  • Tester set to Vickie Lynch

comment:7 Changed 6 years ago by Vickie Lynch

  • Status changed from verifying to closed

Merge remote branch 'origin/bugfix/9748_live_listener_race'

Full changeset: ​26ab2ba39b33f56a69f1f6262e9a961bbb615d93

comment:8 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​10590

Note: See TracTickets for help on using tickets.