Default Project CR-8225

Candidate fix for OPENAM-6864

Closed on 17 Sep 15

  •  
  •  
  •  
  •  
  • Author & Moderator
  • Reviewers
    • Reviewer completed
    • Reviewer completed

CR-8225 6

Keyboard shortcuts  
Summarize the review outcomes (optional)
 
#permalink

Details

Warning: no files are visible, they have all been filtered.
Participant Role Time Spent Comments Latest Comment
Author & Moderator 25m 3 Its like this (an example with am_agent_login and no resp...
Reviewer - Complete 12m 3 ok thats what i thought. i'm done here.
Reviewer - Complete 8m    
Total   44m 6  
#permalink

Objectives

Fix based on the latest core-backtrace file investigation (thanks Nick!). In the event of net-event loop timing out (no response from the remote server), am_net_disconnect is called which signals exit_event, but that (event) might already be signaled and closed by the main thread calling out am_net api (also calling am_net_close/am_net_disconnect).

I've also commented out a list of bootstrap properties from config_xml which are not set from OpenAM so that parser result does not override the local config.

#permalink

Issues Raised From Comments

Key Summary State Assignee
#permalink

General Comments

There are no general comments on this review.
/source/config_xml.c Changed  
Open in IDE #permalink
/source/net_client.c Changed  
Open in IDE #permalink
/source/net_ops.c Changed  
Open in IDE #permalink
/source/thread.c Changed   6
Open in IDE #permalink
/source/thread.h Changed  
Open in IDE #permalink

Review updated: Reload | Ignore | Collapse

You cannot reload the review while writing a comment.

Create Issue

X
Assign To Me

Log time against